feat: add DataVis NITRO condensed view overrides#234
Merged
Conversation
Contributor
garrity-miepub
commented
May 18, 2026
- Reduce table header height (38px → 24px)
- Compact data rows (45px → 39px)
- Tighter title bar, footer, control panel, filter bar
- Smaller field pills, toolbar buttons, group headers
- Affects all DataVis grids when body.condensed is active
- Reduce table header height (38px → 24px) - Compact data rows (45px → 39px) - Tighter title bar, footer, control panel, filter bar - Smaller field pills, toolbar buttons, group headers - Affects all DataVis grids when body.condensed is active
Deploying ui with
|
| Latest commit: |
030b533
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://6baa4fde.ui-6d0.pages.dev |
| Branch Preview URL: | https://feature-datavis-condensed-vi.ui-6d0.pages.dev |
There was a problem hiding this comment.
Pull request overview
Adds condensed-mode CSS overrides for DataVis NITRO grids to increase data density when body.condensed is active.
Changes:
- Introduces a new “DataVis NITRO — Condensed Overrides” section in
condensed-view.css. - Reduces padding/font sizes across DataVis grid areas (title bar, headers, rows/cells, footers, control/filter bars, toolbar, pills, group headers).
Comments suppressed due to low confidence (2)
src/styles/condensed-view.css:11543
wcdv-field-pillappears to be a control (sort/group pill). With the current condensed padding/font-size, its rendered height may drop below the WCAG 2.2 24×24 minimum target size for pointer inputs. If these pills are clickable/draggable controls, add a minimum height (and optionally min-width) to preserve an adequate hit area in condensed mode.
body.condensed .wcdv-field-pill {
padding: 2px 6px !important;
font-size: 0.6875rem !important;
border-radius: 3px !important;
gap: 0.125rem !important;
}
src/styles/condensed-view.css:11559
- Condensed toolbar buttons are reduced to very small padding without an explicit line-height/min-height. This can create sub-24px click targets and make keyboard/pointer interaction harder. Consider adding a
min-height(and possibly min-width) of 24px for.wcdv-toolbar buttonin condensed mode while keeping the visual density.
body.condensed .wcdv-toolbar button {
padding: 2px 6px !important;
font-size: 0.6875rem !important;
}
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or 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
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.