Add pagination to discover embeddable #5770
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: abbyhu2000 <[email protected]>
abbyhu2000
requested review from
ananzh,
kavilla,
AMoo-Miki,
ashwin-pc,
joshuarrrr,
zengyan-amazon,
kristenTian,
zhongnansu,
manasvinibs,
ZilongX,
Flyingliuhub,
BSFishy,
curq,
bandinib-amzn and
SuZhou-Joe
as code owners
AMoo-Miki
reviewed
Feb 1, 2024
| }: DefaultDiscoverTableProps) => { | ||
| const pageSize = 50; |
There was a problem hiding this comment.
I would give anything for this to be changeable in Advanced Settings.
There was a problem hiding this comment.
i think what @ashwin-pc wants is that for now we just copied over how the legacy discover handles it in 2.9.
In 2.9, they just default set pagination for discover embeddable on dashboard page, and non-pagination for discover table on discover page. Also they just do 50 rows per page by default too.
But yea i think we can add those settings in advanced setting in the future too.
There was a problem hiding this comment.
@AMoo-Miki In the new experience this is even easier to configure. The table footer has a drop down that the user can select to choose how many rows they want to display. For the legacy table lets keep the experience the same :)
AMoo-Miki
reviewed
Feb 1, 2024
| const [renderedRowCount, setRenderedRowCount] = useState(50); // Start with 50 rows | ||
| const [displayedRows, setDisplayedRows] = useState( | ||
| showPagination ? rows.slice(0, pageSize) : rows.slice(0, renderedRowCount) |
There was a problem hiding this comment.
We should add the page number to the query string and use it here.
AMoo-Miki
reviewed
Feb 1, 2024
| @@ -81,9 +94,48 @@ export const LegacyDiscoverTable = ({ | |||
| }; | |||
| }, []); | |||
|
|
|||
| const [activePage, setActivePage] = useState(0); | |||
| const PAGE_COUNT = Math.ceil(rows.length / pageSize); | |||
There was a problem hiding this comment.
Suggested change
| const PAGE_COUNT = Math.ceil(rows.length / pageSize); | |
| const pageCount = Math.ceil(rows.length / pageSize); |
Signed-off-by: abbyhu2000 <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feature/table-revert #5770 +/- ##
=====================================================
Coverage 67.02% 67.02%
=====================================================
Files 3296 3296
Lines 63345 63345
Branches 10089 10089
=====================================================
Hits 42458 42458
Misses 18433 18433
Partials 2454 2454
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: abbyhu2000 <[email protected]>
ashwin-pc
reviewed
Feb 1, 2024
| @@ -20,6 +20,7 @@ interface SearchEmbeddableProps { | |||
| } | |||
| export interface DiscoverEmbeddableProps extends DataGridTableProps { | |||
| totalHitCount: number; | |||
| showPagination: boolean; | |||
There was a problem hiding this comment.
Do we need this? DataGridTableProps already has this prop
| </tbody> | ||
| </table> | ||
| {renderedRowCount < rows.length && ( | ||
| {!showPagination && renderedRowCount < rows.length && ( |
There was a problem hiding this comment.
There is message below too that needs to be hidden in the embeddable view right? Or does that not show up?
There was a problem hiding this comment.
Added !showPagination in front of the messages too
| const [renderedRowCount, setRenderedRowCount] = useState(50); // Start with 50 rows | ||
| const [displayedRows, setDisplayedRows] = useState(rows.slice(0, pageSize)); |
There was a problem hiding this comment.
nit: dont perform slicing if its not needed for performance
Suggested change
| const [displayedRows, setDisplayedRows] = useState(rows.slice(0, pageSize)); | |
| const [displayedRows, setDisplayedRows] = useState(showPagination ? rows.slice(0, pageSize) : []); |
There was a problem hiding this comment.
I dont think this is needed since displayedRows is only used when pagination is there.
There was a problem hiding this comment.
Yes but the slice happens even in the discover page. It's an unnecessary slice that can be avoided even if the value isn't used. But it's a bit so we can skip it :)
Comment on lines
+100
to
+103
| const endRow = | ||
| rows.length < pageNumber * pageSize + pageSize | ||
| ? rows.length | ||
| : pageNumber * pageSize + pageSize; |
There was a problem hiding this comment.
nit:
Suggested change
| const endRow = | |
| rows.length < pageNumber * pageSize + pageSize | |
| ? rows.length | |
| : pageNumber * pageSize + pageSize; | |
| const endRow = pageNumber * pageSize + pageSize; |
From MDN for Array.slice
If
end >= array.lengthorendis omitted,array.lengthis used, causing all elements until the end to be extracted.
There was a problem hiding this comment.
Even though the slice case will handle the endRow correctly, i think we still need the endRow to be the correct number for displaying the text such as 1000–1043 of 1043
Signed-off-by: abbyhu2000 <[email protected]>
ashwin-pc
approved these changes
Feb 2, 2024
ashwin-pc
merged commit 40e61ba
into
opensearch-project:feature/table-revert
42 of 65 checks passed
ashwin-pc
added a commit
that referenced
this pull request
Feb 7, 2024
…5789) * basic table working Signed-off-by: abbyhu2000 <[email protected]> * basic styling Signed-off-by: abbyhu2000 <[email protected]> * fix header column style Signed-off-by: abbyhu2000 <[email protected]> * correctly display optional col header actions Signed-off-by: abbyhu2000 <[email protected]> * move col to the right or left if applicable Signed-off-by: abbyhu2000 <[email protected]> * add cell filter for and filter out buttons Signed-off-by: abbyhu2000 <[email protected]> * add cell action on doc table, half working on toggle columns Signed-off-by: abbyhu2000 <[email protected]> * sorting Signed-off-by: abbyhu2000 <[email protected]> * Add toggling between legacy and new table According to #5739, add toggle using storage. Signed-off-by: abbyhu2000 <[email protected]> * get rid of the console log Signed-off-by: abbyhu2000 <[email protected]> * optimize cell filter logics Signed-off-by: abbyhu2000 <[email protected]> * add more refactoring Signed-off-by: abbyhu2000 <[email protected]> * remove an used line Signed-off-by: abbyhu2000 <[email protected]> * some more table cell optimization Signed-off-by: abbyhu2000 <[email protected]> * Initial UI fixes (#5758) Signed-off-by: Ashwin P Chandran <[email protected]> * Table optimization with lazy loading (#5760) Signed-off-by: abbyhu2000 <[email protected]> * [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (#5761) * add a colon (:) after the key * ensure there's a space between each key-value pair * allow feedback url to open in a new tab Signed-off-by: Anan Z <[email protected]> * [Discover] Fix more UI (#5764) Signed-off-by: Ashwin P Chandran <[email protected]> * [Discover] Fixes table optimization (#5763) * [Discover] Fixes table optimization Signed-off-by: Ashwin P Chandran <[email protected]> * improved loader Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * Change sort icon according to current sort conditions (#5772) Signed-off-by: abbyhu2000 <[email protected]> * fix breadcrumb navigation (#5773) Signed-off-by: abbyhu2000 <[email protected]> * Add pagination to discover embeddable (#5770) * Add pagination Signed-off-by: abbyhu2000 <[email protected]> * fix an errir Signed-off-by: abbyhu2000 <[email protected]> * small change Signed-off-by: abbyhu2000 <[email protected]> * fix errors Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * [Discover] Prevent wrapping of time series cells (#5779) Signed-off-by: Miki <[email protected]> * Vertically align the text in QueryStringInputUI with other elements on the page (#5780) Signed-off-by: Miki <[email protected]> * Use `ouiCodeFont` in Discover and reduce text size (#5783) Also: * Fix react `key` errors * vertically align source cells * Clamp the height of the cells * Make sure time-series column never grows Signed-off-by: Miki <[email protected]> * [Discover] Display filter buttons at top right of the cell (#5784) Signed-off-by: Miki <[email protected]> * [Discover] Vertically align the details toggle button (#5785) Signed-off-by: Miki <[email protected]> * [Discover] Fix overflow of the expanded document (#5788) * Fix vertical alignment of expand details button Fix colspan of details cells Signed-off-by: Miki <[email protected]> * Fix overflow problem of detailed doc Signed-off-by: Miki <[email protected]> --------- Signed-off-by: Miki <[email protected]> * update feedback msg (#5787) Signed-off-by: Anan Z <[email protected]> * Resolve sort, default sort and short dot (#5771) Signed-off-by: Anan Z <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> * [Discover] Fixes pagination style (#5778) * fixes pagination style Signed-off-by: Ashwin P Chandran <[email protected]> * style-lint fix Signed-off-by: Ashwin P Chandran <[email protected]> * fix react warning for unnecessary prop Signed-off-by: Ashwin P Chandran <[email protected]> * Show total hit count and pass services in embeddable Signed-off-by: Ashwin P Chandran <[email protected]> * removed unnecessary div Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> * add changelog Signed-off-by: abbyhu2000 <[email protected]> * fix table column error Signed-off-by: abbyhu2000 <[email protected]> * Fix pagination edge case Signed-off-by: abbyhu2000 <[email protected]> * Routing for surrounding doc link should work without a question mark appending to the end (#5776) * routing for surrounding doc should work without ? Signed-off-by: abbyhu2000 <[email protected]> * change path to not include appstate Signed-off-by: abbyhu2000 <[email protected]> * delete optional app state for single doc link too Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * type fixes (#5795) Signed-off-by: Ashwin P Chandran <[email protected]> * Update toggle to button (#5808) Signed-off-by: Ashwin P Chandran <[email protected]> * Adds support for line count (#5814) Signed-off-by: Ashwin P Chandran <[email protected]> * Add swith to datagrid table and fix ciGroup1 (#5816) * add swith to datagrid table and fix ciGroup1 Signed-off-by: Anan <[email protected]> * fix ciGroup7 Signed-off-by: Anan Z <[email protected]> * fix ciGroup6 Signed-off-by: Anan Z <[email protected]> --------- Signed-off-by: Anan <[email protected]> Signed-off-by: Anan Z <[email protected]> * add missing snapshot (#5818) Signed-off-by: Anan Z <[email protected]> * fix pluign ftr tests (#5821) Signed-off-by: Anan Z <[email protected]> * update snapshot (#5824) Signed-off-by: abbyhu2000 <[email protected]> * Fix functional 3 and 4 for discover legacy (#5822) * fix 3 and 4 Signed-off-by: abbyhu2000 <[email protected]> * fix functional 3 and 4 Signed-off-by: abbyhu2000 <[email protected]> * dashboard expect Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * Solve cigroup 7 flakiness (#5826) * test hover Signed-off-by: abbyhu2000 <[email protected]> * comment out other ci to speed up Signed-off-by: abbyhu2000 <[email protected]> * uncomment Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * [Discover] Simplify feedback modal (#5837) * Simplify feedback modal Signed-off-by: Ashwin P Chandran <[email protected]> * renames datagrid settng to newDiscover setting Signed-off-by: Ashwin P Chandran <[email protected]> * removes sub module from branch Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * Persist line height (#5836) Signed-off-by: Ashwin P Chandran <[email protected]> * Update table revert changelog (#5835) * Update table revert changelog ### Description updates changelog ### Issues Resolved <!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix --> <!-- Example: closes #1234 or fixes <Issue_URL> --> ## Screenshot <!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description --> ## Testing the changes <!-- Please provide detailed steps for validating your changes. This could involve specific commands to run, pages to visit, scenarios to try or any other information that would help reviewers verify the functionality of your change --> ### Check List - [ ] All tests pass - [ ] `yarn test:jest` - [ ] `yarn test:jest_integration` - [ ] New functionality includes testing. - [ ] New functionality has been documented. - [ ] Update [CHANGELOG.md](./../CHANGELOG.md) - [ ] Commits are signed per the DCO using --signoff * revert dependency change Signed-off-by: Ashwin P Chandran <[email protected]> * nit Signed-off-by: Ashwin P Chandran <[email protected]> * update changelog Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * fix table height persist on context page Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Anan Z <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Anan <[email protected]> Co-authored-by: Ashwin P Chandran <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Ganesh0107 <[email protected]>
opensearch-trigger-bot bot
pushed a commit
that referenced
this pull request
Feb 7, 2024
…5789) * basic table working Signed-off-by: abbyhu2000 <[email protected]> * basic styling Signed-off-by: abbyhu2000 <[email protected]> * fix header column style Signed-off-by: abbyhu2000 <[email protected]> * correctly display optional col header actions Signed-off-by: abbyhu2000 <[email protected]> * move col to the right or left if applicable Signed-off-by: abbyhu2000 <[email protected]> * add cell filter for and filter out buttons Signed-off-by: abbyhu2000 <[email protected]> * add cell action on doc table, half working on toggle columns Signed-off-by: abbyhu2000 <[email protected]> * sorting Signed-off-by: abbyhu2000 <[email protected]> * Add toggling between legacy and new table According to #5739, add toggle using storage. Signed-off-by: abbyhu2000 <[email protected]> * get rid of the console log Signed-off-by: abbyhu2000 <[email protected]> * optimize cell filter logics Signed-off-by: abbyhu2000 <[email protected]> * add more refactoring Signed-off-by: abbyhu2000 <[email protected]> * remove an used line Signed-off-by: abbyhu2000 <[email protected]> * some more table cell optimization Signed-off-by: abbyhu2000 <[email protected]> * Initial UI fixes (#5758) Signed-off-by: Ashwin P Chandran <[email protected]> * Table optimization with lazy loading (#5760) Signed-off-by: abbyhu2000 <[email protected]> * [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (#5761) * add a colon (:) after the key * ensure there's a space between each key-value pair * allow feedback url to open in a new tab Signed-off-by: Anan Z <[email protected]> * [Discover] Fix more UI (#5764) Signed-off-by: Ashwin P Chandran <[email protected]> * [Discover] Fixes table optimization (#5763) * [Discover] Fixes table optimization Signed-off-by: Ashwin P Chandran <[email protected]> * improved loader Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * Change sort icon according to current sort conditions (#5772) Signed-off-by: abbyhu2000 <[email protected]> * fix breadcrumb navigation (#5773) Signed-off-by: abbyhu2000 <[email protected]> * Add pagination to discover embeddable (#5770) * Add pagination Signed-off-by: abbyhu2000 <[email protected]> * fix an errir Signed-off-by: abbyhu2000 <[email protected]> * small change Signed-off-by: abbyhu2000 <[email protected]> * fix errors Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * [Discover] Prevent wrapping of time series cells (#5779) Signed-off-by: Miki <[email protected]> * Vertically align the text in QueryStringInputUI with other elements on the page (#5780) Signed-off-by: Miki <[email protected]> * Use `ouiCodeFont` in Discover and reduce text size (#5783) Also: * Fix react `key` errors * vertically align source cells * Clamp the height of the cells * Make sure time-series column never grows Signed-off-by: Miki <[email protected]> * [Discover] Display filter buttons at top right of the cell (#5784) Signed-off-by: Miki <[email protected]> * [Discover] Vertically align the details toggle button (#5785) Signed-off-by: Miki <[email protected]> * [Discover] Fix overflow of the expanded document (#5788) * Fix vertical alignment of expand details button Fix colspan of details cells Signed-off-by: Miki <[email protected]> * Fix overflow problem of detailed doc Signed-off-by: Miki <[email protected]> --------- Signed-off-by: Miki <[email protected]> * update feedback msg (#5787) Signed-off-by: Anan Z <[email protected]> * Resolve sort, default sort and short dot (#5771) Signed-off-by: Anan Z <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> * [Discover] Fixes pagination style (#5778) * fixes pagination style Signed-off-by: Ashwin P Chandran <[email protected]> * style-lint fix Signed-off-by: Ashwin P Chandran <[email protected]> * fix react warning for unnecessary prop Signed-off-by: Ashwin P Chandran <[email protected]> * Show total hit count and pass services in embeddable Signed-off-by: Ashwin P Chandran <[email protected]> * removed unnecessary div Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> * add changelog Signed-off-by: abbyhu2000 <[email protected]> * fix table column error Signed-off-by: abbyhu2000 <[email protected]> * Fix pagination edge case Signed-off-by: abbyhu2000 <[email protected]> * Routing for surrounding doc link should work without a question mark appending to the end (#5776) * routing for surrounding doc should work without ? Signed-off-by: abbyhu2000 <[email protected]> * change path to not include appstate Signed-off-by: abbyhu2000 <[email protected]> * delete optional app state for single doc link too Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * type fixes (#5795) Signed-off-by: Ashwin P Chandran <[email protected]> * Update toggle to button (#5808) Signed-off-by: Ashwin P Chandran <[email protected]> * Adds support for line count (#5814) Signed-off-by: Ashwin P Chandran <[email protected]> * Add swith to datagrid table and fix ciGroup1 (#5816) * add swith to datagrid table and fix ciGroup1 Signed-off-by: Anan <[email protected]> * fix ciGroup7 Signed-off-by: Anan Z <[email protected]> * fix ciGroup6 Signed-off-by: Anan Z <[email protected]> --------- Signed-off-by: Anan <[email protected]> Signed-off-by: Anan Z <[email protected]> * add missing snapshot (#5818) Signed-off-by: Anan Z <[email protected]> * fix pluign ftr tests (#5821) Signed-off-by: Anan Z <[email protected]> * update snapshot (#5824) Signed-off-by: abbyhu2000 <[email protected]> * Fix functional 3 and 4 for discover legacy (#5822) * fix 3 and 4 Signed-off-by: abbyhu2000 <[email protected]> * fix functional 3 and 4 Signed-off-by: abbyhu2000 <[email protected]> * dashboard expect Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * Solve cigroup 7 flakiness (#5826) * test hover Signed-off-by: abbyhu2000 <[email protected]> * comment out other ci to speed up Signed-off-by: abbyhu2000 <[email protected]> * uncomment Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * [Discover] Simplify feedback modal (#5837) * Simplify feedback modal Signed-off-by: Ashwin P Chandran <[email protected]> * renames datagrid settng to newDiscover setting Signed-off-by: Ashwin P Chandran <[email protected]> * removes sub module from branch Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * Persist line height (#5836) Signed-off-by: Ashwin P Chandran <[email protected]> * Update table revert changelog (#5835) * Update table revert changelog ### Description updates changelog ### Issues Resolved <!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix --> <!-- Example: closes #1234 or fixes <Issue_URL> --> ## Screenshot <!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description --> ## Testing the changes <!-- Please provide detailed steps for validating your changes. This could involve specific commands to run, pages to visit, scenarios to try or any other information that would help reviewers verify the functionality of your change --> ### Check List - [ ] All tests pass - [ ] `yarn test:jest` - [ ] `yarn test:jest_integration` - [ ] New functionality includes testing. - [ ] New functionality has been documented. - [ ] Update [CHANGELOG.md](./../CHANGELOG.md) - [ ] Commits are signed per the DCO using --signoff * revert dependency change Signed-off-by: Ashwin P Chandran <[email protected]> * nit Signed-off-by: Ashwin P Chandran <[email protected]> * update changelog Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * fix table height persist on context page Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Anan Z <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Anan <[email protected]> Co-authored-by: Ashwin P Chandran <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Ganesh0107 <[email protected]> (cherry picked from commit e13dbff) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
ashwin-pc
pushed a commit
to ashwin-pc/OpenSearch-Dashboards
that referenced
this pull request
Feb 7, 2024
…pensearch-project#5789) * basic table working Signed-off-by: abbyhu2000 <[email protected]> * basic styling Signed-off-by: abbyhu2000 <[email protected]> * fix header column style Signed-off-by: abbyhu2000 <[email protected]> * correctly display optional col header actions Signed-off-by: abbyhu2000 <[email protected]> * move col to the right or left if applicable Signed-off-by: abbyhu2000 <[email protected]> * add cell filter for and filter out buttons Signed-off-by: abbyhu2000 <[email protected]> * add cell action on doc table, half working on toggle columns Signed-off-by: abbyhu2000 <[email protected]> * sorting Signed-off-by: abbyhu2000 <[email protected]> * Add toggling between legacy and new table According to opensearch-project#5739, add toggle using storage. Signed-off-by: abbyhu2000 <[email protected]> * get rid of the console log Signed-off-by: abbyhu2000 <[email protected]> * optimize cell filter logics Signed-off-by: abbyhu2000 <[email protected]> * add more refactoring Signed-off-by: abbyhu2000 <[email protected]> * remove an used line Signed-off-by: abbyhu2000 <[email protected]> * some more table cell optimization Signed-off-by: abbyhu2000 <[email protected]> * Initial UI fixes (opensearch-project#5758) Signed-off-by: Ashwin P Chandran <[email protected]> * Table optimization with lazy loading (opensearch-project#5760) Signed-off-by: abbyhu2000 <[email protected]> * [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (opensearch-project#5761) * add a colon (:) after the key * ensure there's a space between each key-value pair * allow feedback url to open in a new tab Signed-off-by: Anan Z <[email protected]> * [Discover] Fix more UI (opensearch-project#5764) Signed-off-by: Ashwin P Chandran <[email protected]> * [Discover] Fixes table optimization (opensearch-project#5763) * [Discover] Fixes table optimization Signed-off-by: Ashwin P Chandran <[email protected]> * improved loader Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * Change sort icon according to current sort conditions (opensearch-project#5772) Signed-off-by: abbyhu2000 <[email protected]> * fix breadcrumb navigation (opensearch-project#5773) Signed-off-by: abbyhu2000 <[email protected]> * Add pagination to discover embeddable (opensearch-project#5770) * Add pagination Signed-off-by: abbyhu2000 <[email protected]> * fix an errir Signed-off-by: abbyhu2000 <[email protected]> * small change Signed-off-by: abbyhu2000 <[email protected]> * fix errors Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * [Discover] Prevent wrapping of time series cells (opensearch-project#5779) Signed-off-by: Miki <[email protected]> * Vertically align the text in QueryStringInputUI with other elements on the page (opensearch-project#5780) Signed-off-by: Miki <[email protected]> * Use `ouiCodeFont` in Discover and reduce text size (opensearch-project#5783) Also: * Fix react `key` errors * vertically align source cells * Clamp the height of the cells * Make sure time-series column never grows Signed-off-by: Miki <[email protected]> * [Discover] Display filter buttons at top right of the cell (opensearch-project#5784) Signed-off-by: Miki <[email protected]> * [Discover] Vertically align the details toggle button (opensearch-project#5785) Signed-off-by: Miki <[email protected]> * [Discover] Fix overflow of the expanded document (opensearch-project#5788) * Fix vertical alignment of expand details button Fix colspan of details cells Signed-off-by: Miki <[email protected]> * Fix overflow problem of detailed doc Signed-off-by: Miki <[email protected]> --------- Signed-off-by: Miki <[email protected]> * update feedback msg (opensearch-project#5787) Signed-off-by: Anan Z <[email protected]> * Resolve sort, default sort and short dot (opensearch-project#5771) Signed-off-by: Anan Z <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> * [Discover] Fixes pagination style (opensearch-project#5778) * fixes pagination style Signed-off-by: Ashwin P Chandran <[email protected]> * style-lint fix Signed-off-by: Ashwin P Chandran <[email protected]> * fix react warning for unnecessary prop Signed-off-by: Ashwin P Chandran <[email protected]> * Show total hit count and pass services in embeddable Signed-off-by: Ashwin P Chandran <[email protected]> * removed unnecessary div Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> * add changelog Signed-off-by: abbyhu2000 <[email protected]> * fix table column error Signed-off-by: abbyhu2000 <[email protected]> * Fix pagination edge case Signed-off-by: abbyhu2000 <[email protected]> * Routing for surrounding doc link should work without a question mark appending to the end (opensearch-project#5776) * routing for surrounding doc should work without ? Signed-off-by: abbyhu2000 <[email protected]> * change path to not include appstate Signed-off-by: abbyhu2000 <[email protected]> * delete optional app state for single doc link too Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * type fixes (opensearch-project#5795) Signed-off-by: Ashwin P Chandran <[email protected]> * Update toggle to button (opensearch-project#5808) Signed-off-by: Ashwin P Chandran <[email protected]> * Adds support for line count (opensearch-project#5814) Signed-off-by: Ashwin P Chandran <[email protected]> * Add swith to datagrid table and fix ciGroup1 (opensearch-project#5816) * add swith to datagrid table and fix ciGroup1 Signed-off-by: Anan <[email protected]> * fix ciGroup7 Signed-off-by: Anan Z <[email protected]> * fix ciGroup6 Signed-off-by: Anan Z <[email protected]> --------- Signed-off-by: Anan <[email protected]> Signed-off-by: Anan Z <[email protected]> * add missing snapshot (opensearch-project#5818) Signed-off-by: Anan Z <[email protected]> * fix pluign ftr tests (opensearch-project#5821) Signed-off-by: Anan Z <[email protected]> * update snapshot (opensearch-project#5824) Signed-off-by: abbyhu2000 <[email protected]> * Fix functional 3 and 4 for discover legacy (opensearch-project#5822) * fix 3 and 4 Signed-off-by: abbyhu2000 <[email protected]> * fix functional 3 and 4 Signed-off-by: abbyhu2000 <[email protected]> * dashboard expect Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * Solve cigroup 7 flakiness (opensearch-project#5826) * test hover Signed-off-by: abbyhu2000 <[email protected]> * comment out other ci to speed up Signed-off-by: abbyhu2000 <[email protected]> * uncomment Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * [Discover] Simplify feedback modal (opensearch-project#5837) * Simplify feedback modal Signed-off-by: Ashwin P Chandran <[email protected]> * renames datagrid settng to newDiscover setting Signed-off-by: Ashwin P Chandran <[email protected]> * removes sub module from branch Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * Persist line height (opensearch-project#5836) Signed-off-by: Ashwin P Chandran <[email protected]> * Update table revert changelog (opensearch-project#5835) * Update table revert changelog ### Description updates changelog ### Issues Resolved <!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix --> <!-- Example: closes opensearch-project#1234 or fixes <Issue_URL> --> ## Screenshot <!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description --> ## Testing the changes <!-- Please provide detailed steps for validating your changes. This could involve specific commands to run, pages to visit, scenarios to try or any other information that would help reviewers verify the functionality of your change --> ### Check List - [ ] All tests pass - [ ] `yarn test:jest` - [ ] `yarn test:jest_integration` - [ ] New functionality includes testing. - [ ] New functionality has been documented. - [ ] Update [CHANGELOG.md](./../CHANGELOG.md) - [ ] Commits are signed per the DCO using --signoff * revert dependency change Signed-off-by: Ashwin P Chandran <[email protected]> * nit Signed-off-by: Ashwin P Chandran <[email protected]> * update changelog Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * fix table height persist on context page Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Anan Z <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Anan <[email protected]> Co-authored-by: Ashwin P Chandran <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Ganesh0107 <[email protected]> (cherry picked from commit e13dbff)
ashwin-pc
added a commit
that referenced
this pull request
Feb 8, 2024
…5789) (#5840) * basic table working Signed-off-by: abbyhu2000 <[email protected]> * basic styling Signed-off-by: abbyhu2000 <[email protected]> * fix header column style Signed-off-by: abbyhu2000 <[email protected]> * correctly display optional col header actions Signed-off-by: abbyhu2000 <[email protected]> * move col to the right or left if applicable Signed-off-by: abbyhu2000 <[email protected]> * add cell filter for and filter out buttons Signed-off-by: abbyhu2000 <[email protected]> * add cell action on doc table, half working on toggle columns Signed-off-by: abbyhu2000 <[email protected]> * sorting Signed-off-by: abbyhu2000 <[email protected]> * Add toggling between legacy and new table According to #5739, add toggle using storage. Signed-off-by: abbyhu2000 <[email protected]> * get rid of the console log Signed-off-by: abbyhu2000 <[email protected]> * optimize cell filter logics Signed-off-by: abbyhu2000 <[email protected]> * add more refactoring Signed-off-by: abbyhu2000 <[email protected]> * remove an used line Signed-off-by: abbyhu2000 <[email protected]> * some more table cell optimization Signed-off-by: abbyhu2000 <[email protected]> * Initial UI fixes (#5758) Signed-off-by: Ashwin P Chandran <[email protected]> * Table optimization with lazy loading (#5760) Signed-off-by: abbyhu2000 <[email protected]> * [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (#5761) * add a colon (:) after the key * ensure there's a space between each key-value pair * allow feedback url to open in a new tab Signed-off-by: Anan Z <[email protected]> * [Discover] Fix more UI (#5764) Signed-off-by: Ashwin P Chandran <[email protected]> * [Discover] Fixes table optimization (#5763) * [Discover] Fixes table optimization Signed-off-by: Ashwin P Chandran <[email protected]> * improved loader Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * Change sort icon according to current sort conditions (#5772) Signed-off-by: abbyhu2000 <[email protected]> * fix breadcrumb navigation (#5773) Signed-off-by: abbyhu2000 <[email protected]> * Add pagination to discover embeddable (#5770) * Add pagination Signed-off-by: abbyhu2000 <[email protected]> * fix an errir Signed-off-by: abbyhu2000 <[email protected]> * small change Signed-off-by: abbyhu2000 <[email protected]> * fix errors Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * [Discover] Prevent wrapping of time series cells (#5779) Signed-off-by: Miki <[email protected]> * Vertically align the text in QueryStringInputUI with other elements on the page (#5780) Signed-off-by: Miki <[email protected]> * Use `ouiCodeFont` in Discover and reduce text size (#5783) Also: * Fix react `key` errors * vertically align source cells * Clamp the height of the cells * Make sure time-series column never grows Signed-off-by: Miki <[email protected]> * [Discover] Display filter buttons at top right of the cell (#5784) Signed-off-by: Miki <[email protected]> * [Discover] Vertically align the details toggle button (#5785) Signed-off-by: Miki <[email protected]> * [Discover] Fix overflow of the expanded document (#5788) * Fix vertical alignment of expand details button Fix colspan of details cells Signed-off-by: Miki <[email protected]> * Fix overflow problem of detailed doc Signed-off-by: Miki <[email protected]> --------- Signed-off-by: Miki <[email protected]> * update feedback msg (#5787) Signed-off-by: Anan Z <[email protected]> * Resolve sort, default sort and short dot (#5771) Signed-off-by: Anan Z <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> * [Discover] Fixes pagination style (#5778) * fixes pagination style Signed-off-by: Ashwin P Chandran <[email protected]> * style-lint fix Signed-off-by: Ashwin P Chandran <[email protected]> * fix react warning for unnecessary prop Signed-off-by: Ashwin P Chandran <[email protected]> * Show total hit count and pass services in embeddable Signed-off-by: Ashwin P Chandran <[email protected]> * removed unnecessary div Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> * add changelog Signed-off-by: abbyhu2000 <[email protected]> * fix table column error Signed-off-by: abbyhu2000 <[email protected]> * Fix pagination edge case Signed-off-by: abbyhu2000 <[email protected]> * Routing for surrounding doc link should work without a question mark appending to the end (#5776) * routing for surrounding doc should work without ? Signed-off-by: abbyhu2000 <[email protected]> * change path to not include appstate Signed-off-by: abbyhu2000 <[email protected]> * delete optional app state for single doc link too Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * type fixes (#5795) Signed-off-by: Ashwin P Chandran <[email protected]> * Update toggle to button (#5808) Signed-off-by: Ashwin P Chandran <[email protected]> * Adds support for line count (#5814) Signed-off-by: Ashwin P Chandran <[email protected]> * Add swith to datagrid table and fix ciGroup1 (#5816) * add swith to datagrid table and fix ciGroup1 Signed-off-by: Anan <[email protected]> * fix ciGroup7 Signed-off-by: Anan Z <[email protected]> * fix ciGroup6 Signed-off-by: Anan Z <[email protected]> --------- Signed-off-by: Anan <[email protected]> Signed-off-by: Anan Z <[email protected]> * add missing snapshot (#5818) Signed-off-by: Anan Z <[email protected]> * fix pluign ftr tests (#5821) Signed-off-by: Anan Z <[email protected]> * update snapshot (#5824) Signed-off-by: abbyhu2000 <[email protected]> * Fix functional 3 and 4 for discover legacy (#5822) * fix 3 and 4 Signed-off-by: abbyhu2000 <[email protected]> * fix functional 3 and 4 Signed-off-by: abbyhu2000 <[email protected]> * dashboard expect Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * Solve cigroup 7 flakiness (#5826) * test hover Signed-off-by: abbyhu2000 <[email protected]> * comment out other ci to speed up Signed-off-by: abbyhu2000 <[email protected]> * uncomment Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * [Discover] Simplify feedback modal (#5837) * Simplify feedback modal Signed-off-by: Ashwin P Chandran <[email protected]> * renames datagrid settng to newDiscover setting Signed-off-by: Ashwin P Chandran <[email protected]> * removes sub module from branch Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * Persist line height (#5836) Signed-off-by: Ashwin P Chandran <[email protected]> * Update table revert changelog (#5835) * Update table revert changelog ### Description updates changelog ### Issues Resolved <!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix --> <!-- Example: closes #1234 or fixes <Issue_URL> --> ## Screenshot <!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description --> ## Testing the changes <!-- Please provide detailed steps for validating your changes. This could involve specific commands to run, pages to visit, scenarios to try or any other information that would help reviewers verify the functionality of your change --> ### Check List - [ ] All tests pass - [ ] `yarn test:jest` - [ ] `yarn test:jest_integration` - [ ] New functionality includes testing. - [ ] New functionality has been documented. - [ ] Update [CHANGELOG.md](./../CHANGELOG.md) - [ ] Commits are signed per the DCO using --signoff * revert dependency change Signed-off-by: Ashwin P Chandran <[email protected]> * nit Signed-off-by: Ashwin P Chandran <[email protected]> * update changelog Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * fix table height persist on context page Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Anan Z <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Anan <[email protected]> Co-authored-by: Ashwin P Chandran <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Ganesh0107 <[email protected]> (cherry picked from commit e13dbff) Co-authored-by: Qingyang(Abby) Hu <[email protected]>
kavilla
pushed a commit
that referenced
this pull request
Feb 8, 2024
…5789) (#5839) * basic table working * basic styling * fix header column style * correctly display optional col header actions * move col to the right or left if applicable * add cell filter for and filter out buttons * add cell action on doc table, half working on toggle columns * sorting * Add toggling between legacy and new table According to #5739, add toggle using storage. * get rid of the console log * optimize cell filter logics * add more refactoring * remove an used line * some more table cell optimization * Initial UI fixes (#5758) * Table optimization with lazy loading (#5760) * [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (#5761) * add a colon (:) after the key * ensure there's a space between each key-value pair * allow feedback url to open in a new tab * [Discover] Fix more UI (#5764) * [Discover] Fixes table optimization (#5763) * [Discover] Fixes table optimization * improved loader --------- * Change sort icon according to current sort conditions (#5772) * fix breadcrumb navigation (#5773) * Add pagination to discover embeddable (#5770) * Add pagination * fix an errir * small change * fix errors --------- * [Discover] Prevent wrapping of time series cells (#5779) * Vertically align the text in QueryStringInputUI with other elements on the page (#5780) * Use `ouiCodeFont` in Discover and reduce text size (#5783) Also: * Fix react `key` errors * vertically align source cells * Clamp the height of the cells * Make sure time-series column never grows * [Discover] Display filter buttons at top right of the cell (#5784) * [Discover] Vertically align the details toggle button (#5785) * [Discover] Fix overflow of the expanded document (#5788) * Fix vertical alignment of expand details button Fix colspan of details cells * Fix overflow problem of detailed doc --------- * update feedback msg (#5787) * Resolve sort, default sort and short dot (#5771) * [Discover] Fixes pagination style (#5778) * fixes pagination style * style-lint fix * fix react warning for unnecessary prop * Show total hit count and pass services in embeddable * removed unnecessary div --------- * add changelog * fix table column error * Fix pagination edge case * Routing for surrounding doc link should work without a question mark appending to the end (#5776) * routing for surrounding doc should work without ? * change path to not include appstate * delete optional app state for single doc link too --------- * type fixes (#5795) * Update toggle to button (#5808) * Adds support for line count (#5814) * Add swith to datagrid table and fix ciGroup1 (#5816) * add swith to datagrid table and fix ciGroup1 * fix ciGroup7 * fix ciGroup6 --------- * add missing snapshot (#5818) * fix pluign ftr tests (#5821) * update snapshot (#5824) * Fix functional 3 and 4 for discover legacy (#5822) * fix 3 and 4 * fix functional 3 and 4 * dashboard expect --------- * Solve cigroup 7 flakiness (#5826) * test hover * comment out other ci to speed up * uncomment --------- * [Discover] Simplify feedback modal (#5837) * Simplify feedback modal * renames datagrid settng to newDiscover setting * removes sub module from branch --------- * Persist line height (#5836) * Update table revert changelog (#5835) * Update table revert changelog ### Description updates changelog ### Issues Resolved <!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix --> <!-- Example: closes #1234 or fixes <Issue_URL> --> ## Screenshot <!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description --> ## Testing the changes <!-- Please provide detailed steps for validating your changes. This could involve specific commands to run, pages to visit, scenarios to try or any other information that would help reviewers verify the functionality of your change --> ### Check List - [ ] All tests pass - [ ] `yarn test:jest` - [ ] `yarn test:jest_integration` - [ ] New functionality includes testing. - [ ] New functionality has been documented. - [ ] Update [CHANGELOG.md](./../CHANGELOG.md) - [ ] Commits are signed per the DCO using --signoff * revert dependency change * nit * update changelog --------- * fix table height persist on context page --------- (cherry picked from commit e13dbff) Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Anan Z <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Anan <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Ashwin P Chandran <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Ganesh0107 <[email protected]>
ananzh
added a commit
to ananzh/OpenSearch-Dashboards
that referenced
this pull request
Feb 21, 2024
…pensearch-project#5789) * basic table working Signed-off-by: abbyhu2000 <[email protected]> * basic styling Signed-off-by: abbyhu2000 <[email protected]> * fix header column style Signed-off-by: abbyhu2000 <[email protected]> * correctly display optional col header actions Signed-off-by: abbyhu2000 <[email protected]> * move col to the right or left if applicable Signed-off-by: abbyhu2000 <[email protected]> * add cell filter for and filter out buttons Signed-off-by: abbyhu2000 <[email protected]> * add cell action on doc table, half working on toggle columns Signed-off-by: abbyhu2000 <[email protected]> * sorting Signed-off-by: abbyhu2000 <[email protected]> * Add toggling between legacy and new table According to opensearch-project#5739, add toggle using storage. Signed-off-by: abbyhu2000 <[email protected]> * get rid of the console log Signed-off-by: abbyhu2000 <[email protected]> * optimize cell filter logics Signed-off-by: abbyhu2000 <[email protected]> * add more refactoring Signed-off-by: abbyhu2000 <[email protected]> * remove an used line Signed-off-by: abbyhu2000 <[email protected]> * some more table cell optimization Signed-off-by: abbyhu2000 <[email protected]> * Initial UI fixes (opensearch-project#5758) Signed-off-by: Ashwin P Chandran <[email protected]> * Table optimization with lazy loading (opensearch-project#5760) Signed-off-by: abbyhu2000 <[email protected]> * [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (opensearch-project#5761) * add a colon (:) after the key * ensure there's a space between each key-value pair * allow feedback url to open in a new tab Signed-off-by: Anan Z <[email protected]> * [Discover] Fix more UI (opensearch-project#5764) Signed-off-by: Ashwin P Chandran <[email protected]> * [Discover] Fixes table optimization (opensearch-project#5763) * [Discover] Fixes table optimization Signed-off-by: Ashwin P Chandran <[email protected]> * improved loader Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * Change sort icon according to current sort conditions (opensearch-project#5772) Signed-off-by: abbyhu2000 <[email protected]> * fix breadcrumb navigation (opensearch-project#5773) Signed-off-by: abbyhu2000 <[email protected]> * Add pagination to discover embeddable (opensearch-project#5770) * Add pagination Signed-off-by: abbyhu2000 <[email protected]> * fix an errir Signed-off-by: abbyhu2000 <[email protected]> * small change Signed-off-by: abbyhu2000 <[email protected]> * fix errors Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * [Discover] Prevent wrapping of time series cells (opensearch-project#5779) Signed-off-by: Miki <[email protected]> * Vertically align the text in QueryStringInputUI with other elements on the page (opensearch-project#5780) Signed-off-by: Miki <[email protected]> * Use `ouiCodeFont` in Discover and reduce text size (opensearch-project#5783) Also: * Fix react `key` errors * vertically align source cells * Clamp the height of the cells * Make sure time-series column never grows Signed-off-by: Miki <[email protected]> * [Discover] Display filter buttons at top right of the cell (opensearch-project#5784) Signed-off-by: Miki <[email protected]> * [Discover] Vertically align the details toggle button (opensearch-project#5785) Signed-off-by: Miki <[email protected]> * [Discover] Fix overflow of the expanded document (opensearch-project#5788) * Fix vertical alignment of expand details button Fix colspan of details cells Signed-off-by: Miki <[email protected]> * Fix overflow problem of detailed doc Signed-off-by: Miki <[email protected]> --------- Signed-off-by: Miki <[email protected]> * update feedback msg (opensearch-project#5787) Signed-off-by: Anan Z <[email protected]> * Resolve sort, default sort and short dot (opensearch-project#5771) Signed-off-by: Anan Z <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> * [Discover] Fixes pagination style (opensearch-project#5778) * fixes pagination style Signed-off-by: Ashwin P Chandran <[email protected]> * style-lint fix Signed-off-by: Ashwin P Chandran <[email protected]> * fix react warning for unnecessary prop Signed-off-by: Ashwin P Chandran <[email protected]> * Show total hit count and pass services in embeddable Signed-off-by: Ashwin P Chandran <[email protected]> * removed unnecessary div Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> * add changelog Signed-off-by: abbyhu2000 <[email protected]> * fix table column error Signed-off-by: abbyhu2000 <[email protected]> * Fix pagination edge case Signed-off-by: abbyhu2000 <[email protected]> * Routing for surrounding doc link should work without a question mark appending to the end (opensearch-project#5776) * routing for surrounding doc should work without ? Signed-off-by: abbyhu2000 <[email protected]> * change path to not include appstate Signed-off-by: abbyhu2000 <[email protected]> * delete optional app state for single doc link too Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * type fixes (opensearch-project#5795) Signed-off-by: Ashwin P Chandran <[email protected]> * Update toggle to button (opensearch-project#5808) Signed-off-by: Ashwin P Chandran <[email protected]> * Adds support for line count (opensearch-project#5814) Signed-off-by: Ashwin P Chandran <[email protected]> * Add swith to datagrid table and fix ciGroup1 (opensearch-project#5816) * add swith to datagrid table and fix ciGroup1 Signed-off-by: Anan <[email protected]> * fix ciGroup7 Signed-off-by: Anan Z <[email protected]> * fix ciGroup6 Signed-off-by: Anan Z <[email protected]> --------- Signed-off-by: Anan <[email protected]> Signed-off-by: Anan Z <[email protected]> * add missing snapshot (opensearch-project#5818) Signed-off-by: Anan Z <[email protected]> * fix pluign ftr tests (opensearch-project#5821) Signed-off-by: Anan Z <[email protected]> * update snapshot (opensearch-project#5824) Signed-off-by: abbyhu2000 <[email protected]> * Fix functional 3 and 4 for discover legacy (opensearch-project#5822) * fix 3 and 4 Signed-off-by: abbyhu2000 <[email protected]> * fix functional 3 and 4 Signed-off-by: abbyhu2000 <[email protected]> * dashboard expect Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * Solve cigroup 7 flakiness (opensearch-project#5826) * test hover Signed-off-by: abbyhu2000 <[email protected]> * comment out other ci to speed up Signed-off-by: abbyhu2000 <[email protected]> * uncomment Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * [Discover] Simplify feedback modal (opensearch-project#5837) * Simplify feedback modal Signed-off-by: Ashwin P Chandran <[email protected]> * renames datagrid settng to newDiscover setting Signed-off-by: Ashwin P Chandran <[email protected]> * removes sub module from branch Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * Persist line height (opensearch-project#5836) Signed-off-by: Ashwin P Chandran <[email protected]> * Update table revert changelog (opensearch-project#5835) * Update table revert changelog updates changelog <!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix --> <!-- Example: closes opensearch-project#1234 or fixes <Issue_URL> --> <!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description --> <!-- Please provide detailed steps for validating your changes. This could involve specific commands to run, pages to visit, scenarios to try or any other information that would help reviewers verify the functionality of your change --> - [ ] All tests pass - [ ] `yarn test:jest` - [ ] `yarn test:jest_integration` - [ ] New functionality includes testing. - [ ] New functionality has been documented. - [ ] Update [CHANGELOG.md](./../CHANGELOG.md) - [ ] Commits are signed per the DCO using --signoff * revert dependency change Signed-off-by: Ashwin P Chandran <[email protected]> * nit Signed-off-by: Ashwin P Chandran <[email protected]> * update changelog Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * fix table height persist on context page Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Anan Z <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Anan <[email protected]> Co-authored-by: Ashwin P Chandran <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Ganesh0107 <[email protected]>
ananzh
added a commit
to ananzh/OpenSearch-Dashboards
that referenced
this pull request
Feb 21, 2024
…pensearch-project#5789) * basic table working Signed-off-by: abbyhu2000 <[email protected]> * basic styling Signed-off-by: abbyhu2000 <[email protected]> * fix header column style Signed-off-by: abbyhu2000 <[email protected]> * correctly display optional col header actions Signed-off-by: abbyhu2000 <[email protected]> * move col to the right or left if applicable Signed-off-by: abbyhu2000 <[email protected]> * add cell filter for and filter out buttons Signed-off-by: abbyhu2000 <[email protected]> * add cell action on doc table, half working on toggle columns Signed-off-by: abbyhu2000 <[email protected]> * sorting Signed-off-by: abbyhu2000 <[email protected]> * Add toggling between legacy and new table According to opensearch-project#5739, add toggle using storage. Signed-off-by: abbyhu2000 <[email protected]> * get rid of the console log Signed-off-by: abbyhu2000 <[email protected]> * optimize cell filter logics Signed-off-by: abbyhu2000 <[email protected]> * add more refactoring Signed-off-by: abbyhu2000 <[email protected]> * remove an used line Signed-off-by: abbyhu2000 <[email protected]> * some more table cell optimization Signed-off-by: abbyhu2000 <[email protected]> * Initial UI fixes (opensearch-project#5758) Signed-off-by: Ashwin P Chandran <[email protected]> * Table optimization with lazy loading (opensearch-project#5760) Signed-off-by: abbyhu2000 <[email protected]> * [Discover with Legacy Table] fix copy issue and open feedbacks in a new tab (opensearch-project#5761) * add a colon (:) after the key * ensure there's a space between each key-value pair * allow feedback url to open in a new tab Signed-off-by: Anan Z <[email protected]> * [Discover] Fix more UI (opensearch-project#5764) Signed-off-by: Ashwin P Chandran <[email protected]> * [Discover] Fixes table optimization (opensearch-project#5763) * [Discover] Fixes table optimization Signed-off-by: Ashwin P Chandran <[email protected]> * improved loader Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * Change sort icon according to current sort conditions (opensearch-project#5772) Signed-off-by: abbyhu2000 <[email protected]> * fix breadcrumb navigation (opensearch-project#5773) Signed-off-by: abbyhu2000 <[email protected]> * Add pagination to discover embeddable (opensearch-project#5770) * Add pagination Signed-off-by: abbyhu2000 <[email protected]> * fix an errir Signed-off-by: abbyhu2000 <[email protected]> * small change Signed-off-by: abbyhu2000 <[email protected]> * fix errors Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * [Discover] Prevent wrapping of time series cells (opensearch-project#5779) Signed-off-by: Miki <[email protected]> * Vertically align the text in QueryStringInputUI with other elements on the page (opensearch-project#5780) Signed-off-by: Miki <[email protected]> * Use `ouiCodeFont` in Discover and reduce text size (opensearch-project#5783) Also: * Fix react `key` errors * vertically align source cells * Clamp the height of the cells * Make sure time-series column never grows Signed-off-by: Miki <[email protected]> * [Discover] Display filter buttons at top right of the cell (opensearch-project#5784) Signed-off-by: Miki <[email protected]> * [Discover] Vertically align the details toggle button (opensearch-project#5785) Signed-off-by: Miki <[email protected]> * [Discover] Fix overflow of the expanded document (opensearch-project#5788) * Fix vertical alignment of expand details button Fix colspan of details cells Signed-off-by: Miki <[email protected]> * Fix overflow problem of detailed doc Signed-off-by: Miki <[email protected]> --------- Signed-off-by: Miki <[email protected]> * update feedback msg (opensearch-project#5787) Signed-off-by: Anan Z <[email protected]> * Resolve sort, default sort and short dot (opensearch-project#5771) Signed-off-by: Anan Z <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> * [Discover] Fixes pagination style (opensearch-project#5778) * fixes pagination style Signed-off-by: Ashwin P Chandran <[email protected]> * style-lint fix Signed-off-by: Ashwin P Chandran <[email protected]> * fix react warning for unnecessary prop Signed-off-by: Ashwin P Chandran <[email protected]> * Show total hit count and pass services in embeddable Signed-off-by: Ashwin P Chandran <[email protected]> * removed unnecessary div Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> * add changelog Signed-off-by: abbyhu2000 <[email protected]> * fix table column error Signed-off-by: abbyhu2000 <[email protected]> * Fix pagination edge case Signed-off-by: abbyhu2000 <[email protected]> * Routing for surrounding doc link should work without a question mark appending to the end (opensearch-project#5776) * routing for surrounding doc should work without ? Signed-off-by: abbyhu2000 <[email protected]> * change path to not include appstate Signed-off-by: abbyhu2000 <[email protected]> * delete optional app state for single doc link too Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * type fixes (opensearch-project#5795) Signed-off-by: Ashwin P Chandran <[email protected]> * Update toggle to button (opensearch-project#5808) Signed-off-by: Ashwin P Chandran <[email protected]> * Adds support for line count (opensearch-project#5814) Signed-off-by: Ashwin P Chandran <[email protected]> * Add swith to datagrid table and fix ciGroup1 (opensearch-project#5816) * add swith to datagrid table and fix ciGroup1 Signed-off-by: Anan <[email protected]> * fix ciGroup7 Signed-off-by: Anan Z <[email protected]> * fix ciGroup6 Signed-off-by: Anan Z <[email protected]> --------- Signed-off-by: Anan <[email protected]> Signed-off-by: Anan Z <[email protected]> * add missing snapshot (opensearch-project#5818) Signed-off-by: Anan Z <[email protected]> * fix pluign ftr tests (opensearch-project#5821) Signed-off-by: Anan Z <[email protected]> * update snapshot (opensearch-project#5824) Signed-off-by: abbyhu2000 <[email protected]> * Fix functional 3 and 4 for discover legacy (opensearch-project#5822) * fix 3 and 4 Signed-off-by: abbyhu2000 <[email protected]> * fix functional 3 and 4 Signed-off-by: abbyhu2000 <[email protected]> * dashboard expect Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * Solve cigroup 7 flakiness (opensearch-project#5826) * test hover Signed-off-by: abbyhu2000 <[email protected]> * comment out other ci to speed up Signed-off-by: abbyhu2000 <[email protected]> * uncomment Signed-off-by: abbyhu2000 <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> * [Discover] Simplify feedback modal (opensearch-project#5837) * Simplify feedback modal Signed-off-by: Ashwin P Chandran <[email protected]> * renames datagrid settng to newDiscover setting Signed-off-by: Ashwin P Chandran <[email protected]> * removes sub module from branch Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * Persist line height (opensearch-project#5836) Signed-off-by: Ashwin P Chandran <[email protected]> * Update table revert changelog (opensearch-project#5835) * Update table revert changelog updates changelog <!-- List any issues this PR will resolve. Prefix the issue with the keyword closes, fixes, fix --> <!-- Example: closes opensearch-project#1234 or fixes <Issue_URL> --> <!-- Attach any relevant screenshots. Any change to the UI requires an attached screenshot in the PR Description --> <!-- Please provide detailed steps for validating your changes. This could involve specific commands to run, pages to visit, scenarios to try or any other information that would help reviewers verify the functionality of your change --> - [ ] All tests pass - [ ] `yarn test:jest` - [ ] `yarn test:jest_integration` - [ ] New functionality includes testing. - [ ] New functionality has been documented. - [ ] Update [CHANGELOG.md](./../CHANGELOG.md) - [ ] Commits are signed per the DCO using --signoff * revert dependency change Signed-off-by: Ashwin P Chandran <[email protected]> * nit Signed-off-by: Ashwin P Chandran <[email protected]> * update changelog Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: Ashwin P Chandran <[email protected]> * fix table height persist on context page Signed-off-by: Ashwin P Chandran <[email protected]> --------- Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Anan Z <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Anan <[email protected]> Co-authored-by: Ashwin P Chandran <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Ganesh0107 <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Add pagination to discover embeddable that is shown on the dashboard.
According to 2.9 legacy discover:
Issues Resolved
Screenshot
Screen.Recording.2024-02-01.at.12.07.04.PM.mov
Testing the changes
Check List
yarn test:jestyarn test:jest_integration