-
Notifications
You must be signed in to change notification settings - Fork 75
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add table column setting to Environments page #2789
feat: add table column setting to Environments page #2789
Conversation
Your org requires the Graphite merge queue for merging into mainAdd the label “flow:merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “flow:hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @agatha197 and the rest of your teammates on Graphite |
cf42f0e
to
09bf733
Compare
2b8056f
to
1dbe343
Compare
09bf733
to
a98a494
Compare
d8c4775
to
412bd1c
Compare
01fc2fd
to
047ba54
Compare
412bd1c
to
dab23a0
Compare
047ba54
to
6695815
Compare
dab23a0
to
0011774
Compare
6695815
to
cbb0e50
Compare
0011774
to
fde8595
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
cbb0e50
to
e1139c6
Compare
fde8595
to
ede0978
Compare
2346c51
to
55bcf7e
Compare
b8a0b37
to
a980e40
Compare
70a122e
to
58212b1
Compare
92cdd3e
to
143e45d
Compare
58212b1
to
c55d292
Compare
143e45d
to
2ba59aa
Compare
c55d292
to
23f8a66
Compare
2ba59aa
to
7dbc971
Compare
23f8a66
to
232381e
Compare
408ee75
to
d7d64c4
Compare
232381e
to
7662c65
Compare
Merge activity
|
7662c65
to
53af4e3
Compare
**Changes:** This PR enhances the ImageList component in the following ways: 1. Adds a new TableColumnsSettingModal component for customizing visible columns. 2. Implements column visibility persistence using local storage. 3. Introduces a settings button to open the column customization modal. 4. Improves the digest column rendering with ellipsis and tooltip for better readability. **How to test:** > endpoint: 10.100.64.15 1. Checkout core branch to `topic/10-22-feature_add_info_field_to_gql_image_schema` 2. Visit Environments page. 3. Check the setting icon at the bottom of the table. **Rationale:** These changes provide users with more control over the information displayed in the image list, improving the overall user experience and allowing for a more personalized view of the data. **Effects:** - Users can now customize which columns are visible in the image list. - Column visibility preferences are persisted across sessions. - Long digest values are now truncated with an ellipsis and full text available via tooltip. **Screenshots:** ![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/e85e86c3-85c9-4137-9571-d707f6ae4fce.png) **Checklist:** - [ ] Documentation: Update user guide to explain the new column customization feature - [ ] Test case: Verify that column visibility settings are correctly saved and applied - [ ] Test case: Ensure the digest column truncation and tooltip work as expected
53af4e3
to
813b937
Compare
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
---|---|---|---|
🔴 | Statements | 4.92% (-0% 🔻) |
349/7096 |
🔴 | Branches | 4.54% (-0% 🔻) |
219/4828 |
🔴 | Functions | 2.85% (-0% 🔻) |
67/2350 |
🔴 | Lines | 4.81% (-0% 🔻) |
334/6942 |
Test suite run success
93 tests passing in 12 suites.
Report generated by 🧪jest coverage report action from 813b937
Changes:
This PR enhances the ImageList component in the following ways:
How to test:
topic/10-22-feature_add_info_field_to_gql_image_schema
Rationale:
These changes provide users with more control over the information displayed in the image list, improving the overall user experience and allowing for a more personalized view of the data.
Effects:
Screenshots:
Checklist: