chore: reformat all css files#4534
Merged
GordonSmith merged 1 commit intohpcc-systems:mainfrom Feb 26, 2026
Merged
Conversation
Signed-off-by: Gordon Smith <GordonJSmith@gmail.com>
GordonSmith
force-pushed
the
REFORMAT_CSS
branch
from
b130a91 to
d83af95
Compare
There was a problem hiding this comment.
Pull request overview
This PR performs a repo-wide CSS reformat/normalization across many HPCC Viz Framework packages to make styling sources consistent and easier to maintain.
Changes:
- Standardize indentation/whitespace and selector formatting (spacing around combinators, brace placement, blank lines).
- Normalize value formatting (e.g.,
0.25vs.25,1pxvs1.0px, consistent hex casing, multi-linebox-shadow/font). - Minor structural formatting changes (e.g., splitting single-line rules into multi-line blocks).
Reviewed changes
Copilot reviewed 36 out of 83 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/tree/src/Treemap.css | Reformat treemap styles (indentation/selector spacing). |
| packages/tree/src/SunburstPartition.css | Normalize whitespace in sunburst partition styles. |
| packages/tree/src/Dendrogram.css | Reformat dendrogram styles and numeric formatting. |
| packages/tree/src/CirclePacking.css | Normalize numeric values and whitespace. |
| packages/timeline/src/MiniGantt.css | Reformat selectors/spacing for MiniGantt styles. |
| packages/phosphor/src/WidgetAdapter.css | Normalize colors/box-shadow formatting. |
| packages/phosphor/src/DockPanel.css | Normalize colors, quoting, and multi-line font declaration. |
| packages/other/src/ThemeEditor.css | Large-scale formatting normalization of ThemeEditor styles. |
| packages/other/src/Table.css | Reformat table styles; normalize selector grouping/spacing. |
| packages/other/src/Select.css | Reformat selectors and whitespace. |
| packages/other/src/RadioCheckbox.css | Reformat selectors and whitespace. |
| packages/other/src/PropertyEditor.css | Large-scale formatting normalization of PropertyEditor styles. |
| packages/other/src/Paginator.css | Reformat selector spacing and layout. |
| packages/other/src/Opportunity.css | Reformat graph opportunity styles (spacing/value formatting). |
| packages/other/src/MorphText.css | Minor brace/whitespace normalization. |
| packages/other/src/Legend.css | Reformat legend/table-related styles. |
| packages/other/src/IconList.css | Reformat and normalize indentation. |
| packages/other/src/Html.css | Normalize spacing/indentation. |
| packages/other/src/CalendarHeatMap.css | Normalize indentation/whitespace and closing braces. |
| packages/other/src/AutoCompleteText.css | Reformat multi-property blocks into multi-line declarations. |
| packages/observablehq-compiler/index.css | Normalize indentation for compiler placeholder styles. |
| packages/markdown-it-plugins/tests/style.css | Normalize indentation/whitespace in test styles. |
| packages/markdown-it-plugins/src/vitepress/style.css | Reformat vitepress theme variables and color-mix line breaks. |
| packages/markdown-it-plugins/src/vitepress/plot.css | Normalize indentation/whitespace. |
| packages/markdown-it-plugins/src/vitepress/note.css | Reformat note/tip/warning blocks and color-mix formatting. |
| packages/markdown-it-plugins/src/vitepress/layout.css | Large-scale formatting normalization (indentation, multi-line gradients, transitions). |
| packages/markdown-it-plugins/src/vitepress/inspector.css | Normalize indentation and block formatting. |
| packages/markdown-it-plugins/src/vitepress/grid.css | Normalize indentation, container queries formatting. |
| packages/markdown-it-plugins/src/vitepress/global.css | Normalize indentation and wrap long variable lists. |
| packages/markdown-it-plugins/src/vitepress/card.css | Normalize indentation/whitespace. |
| packages/map/src/leaflet/plugins/D3SvgOverlay.css | Normalize indentation/whitespace. |
| packages/map/src/leaflet/plugins/BeautifyIcon.css | Normalize indentation and selector spacing. |
| packages/map/src/leaflet/World.css | Normalize indentation/whitespace. |
| packages/map/src/leaflet/Leaflet.css | Normalize rgba formatting and indentation. |
| packages/map/src/leaflet/ClusterCircles.css | Reformat font declarations and nested selector spacing. |
| packages/map/src/Pins.css | Normalize selector spacing and indentation. |
| packages/map/src/OpenStreet.css | Normalize numeric formatting (0.5px, 0.5). |
| packages/map/src/Lines.css | Normalize indentation and trailing whitespace. |
| packages/map/src/Layered.css | Normalize indentation and add missing semicolon formatting. |
| packages/map/src/Graticule.css | Normalize closing brace formatting. |
| packages/map/src/Graph.css | Normalize indentation and numeric formatting. |
| packages/map/src/GeoHash.css | Normalize indentation/selector spacing. |
| packages/map/src/GMap.css | Normalize formatting and expand one-liners. |
| packages/map/src/Choropleth.css | Normalize indentation, hex casing, and numeric formatting. |
| packages/layout/src/Toolbar.css | Normalize indentation/spacing; convert empty rule to explicit form. |
| packages/layout/src/Tabbed.css | Normalize indentation and spacing. |
| packages/layout/src/Surface.css | Normalize indentation and expand one-line pseudo-class blocks. |
| packages/layout/src/Modal.css | Normalize indentation and spacing. |
| packages/layout/src/Layered.css | Normalize selector spacing (> combinators) and indentation. |
| packages/layout/src/Grid.css | Normalize indentation and expand long gradients. |
| packages/layout/src/FlexGrid.css | Normalize spacing and add blank line separation. |
| packages/layout/src/ChartPanel.css | Normalize indentation and !important spacing. |
| packages/layout/src/Cell.css | Normalize indentation and brace placement. |
| packages/layout/src/Carousel.css | Normalize selector spacing and indentation. |
| packages/layout/src/Border2.css | Normalize selector spacing and indentation. |
| packages/layout/src/Border.css | Large-scale formatting normalization (attribute selectors, gradients). |
| packages/layout/src/Accordion.css | Large-scale formatting normalization (nest-like selectors, spacing). |
| packages/layout/src/AbsoluteSurface.css | Normalize selector spacing and indentation. |
| packages/html/src/TitleBar.css | Normalize selector grouping and indentation. |
| packages/graph/src/common/sankeyGraph.css | Normalize numeric formatting and indentation. |
| packages/graph/src/common/graphT.css | Normalize indentation and focus outline formatting. |
| packages/graph/src/Vertex.css | Normalize indentation/spacing. |
| packages/graph/src/Subgraph.css | Normalize numeric formatting and selector spacing. |
| packages/graph/src/Sankey.css | Normalize numeric formatting and indentation. |
| packages/graph/src/Graph.css | Normalize numeric formatting and indentation. |
| packages/graph/src/Edge.css | Normalize numeric formatting and indentation. |
| packages/form/src/Slider.css | Normalize indentation and block structure. |
| packages/form/src/OnOff.css | Reformat into multi-line declarations and normalize selector spacing. |
| packages/form/src/Input.css | Normalize indentation and selector spacing. |
| packages/form/src/Form.css | Normalize indentation, spacing, and hex casing. |
| packages/eclwatch/src/WUGraphLegend.css | Normalize indentation/selector spacing. |
| packages/eclwatch/src/WUGraph.css | Normalize indentation and numeric formatting. |
| packages/dgrid2/src/table.css | Normalize quoting and indentation. |
| packages/dgrid/src/Common.css | Large-scale formatting normalization; normalize spacing and comments. |
| packages/composite/src/MegaChart.css | Normalize selector spacing and indentation. |
| packages/composite/src/Dermatology.css | Normalize indentation and spacing. |
| packages/composite/src/ChartPanel.css | Normalize selector spacing and indentation. |
| packages/common/tests/coreTest.css | Normalize indentation and whitespace. |
| packages/common/src/Widget.css | Normalize indentation. |
| packages/common/src/TitleBar.css | Normalize indentation and spacing; expand/align comments. |
| packages/common/src/TextBox.css | Normalize indentation/whitespace. |
| packages/common/src/Text.css | Normalize indentation/whitespace. |
| packages/common/src/Surface.css | Normalize indentation and expand pseudo-class blocks. |
| packages/common/src/Shape.css | Normalize indentation/whitespace. |
| packages/common/src/SVGZoomWidget.css | Normalize indentation and closing braces. |
| packages/common/src/ResizeSurface.css | Normalize indentation and closing braces. |
| packages/common/src/Menu.css | Normalize indentation and closing braces. |
| packages/common/src/List.css | Normalize indentation and closing braces. |
| packages/common/src/Icon.css | Normalize indentation and add blank line separation. |
| packages/common/src/FAChar.css | Normalize indentation/whitespace. |
| packages/common/src/EntityRect.css | Normalize indentation/selector spacing. |
| packages/codemirror/src/Editor.css | Normalize indentation and long data-URL formatting. |
| packages/codemirror/src/ECLEditor.css | Expand one-line rules into multi-line blocks; normalize indentation. |
| packages/chart/src/XYAxis.css | Normalize selector spacing and numeric formatting. |
| packages/chart/src/WordCloud.css | Normalize indentation/whitespace. |
| packages/chart/src/Summary.css | Normalize indentation and whitespace. |
| packages/chart/src/Scatter.css | Normalize indentation and whitespace. |
| packages/chart/src/RadialBar.css | Normalize indentation and add missing semicolons. |
| packages/chart/src/Radar.css | Normalize indentation and closing braces. |
| packages/chart/src/Pie.css | Normalize indentation and numeric formatting. |
| packages/chart/src/Line.css | Normalize closing brace and remove trailing blank line. |
| packages/chart/src/HexBin.css | Normalize numeric formatting and closing braces. |
| packages/chart/src/Column.css | Normalize indentation and whitespace. |
| packages/chart/src/Bullet.css | Normalize indentation and numeric formatting. |
| packages/chart/src/Bubble.css | Normalize indentation and add missing semicolons. |
| packages/chart/src/Axis.css | Normalize selector spacing and brace placement. |
| packages/api/src/ITooltip.css | Normalize indentation, spacing, and hex casing. |
Comments suppressed due to low confidence (1)
packages/html/src/TitleBar.css:86
color: noneis not a valid CSS value, so this declaration is ignored. If the intent is to remove an explicit color, usecolor: inheritor delete the property.
| border-width: 2px; | ||
| border-radius: 8px; | ||
| border-color:"black"; | ||
| border-color: "black"; |
There was a problem hiding this comment.
border-color value is quoted, which makes the declaration invalid CSS and the border color won’t apply. Use an unquoted color value (e.g., black) or a hex/rgb value instead.
Suggested change
| border-color: "black"; | |
| border-color: black; |
Comment on lines
48
to
50
| border-color: darkGray; | ||
| background-color:none; | ||
| background-color: none; | ||
| position: absolute; |
There was a problem hiding this comment.
background-color: none is not a valid CSS value, so this declaration is ignored. Use transparent (or remove the declaration) if the intent is “no background color”.
Comment on lines
9
to
11
| outline: 0; | ||
| border-radius: 100; | ||
| } |
There was a problem hiding this comment.
border-radius: 100 is missing a unit, making the declaration invalid (non-zero lengths require units). Use 100px/100% (or another intended unit) to ensure the rounding applies.
Comment on lines
65
to
67
| .graph_Opportunity .arrowhead { | ||
| refX:6; | ||
| refX: 6; | ||
| } |
There was a problem hiding this comment.
refx is not a valid CSS property, so this rule has no effect. If the goal is to set the SVG marker’s refX, it needs to be set as an attribute on the <marker> element (or via JS), not via CSS.
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.
Checklist:
Testing: