Geo - Move SCSS to page_bundles
What does this MR do and why?
Closes #366610 (closed)
This change moves the custom Geo SCSS to their respected page_bundles. I took the initiative to break the styles between each page. If it is better to keep one geo.scss page bundle I can revert them back into one file included in all geo pages.
Files created in this MR:
1. legacy_geo.scss => Used on /admin/geo/replicable/projects and can be removed as part of &4623 (closed)
2. geo_nodes.scss => Used for /admin/geo/sites
3. geo_form.scss => Used for /admin/geo/sites/new and /admin/geo/sites/${id}/edit
4. geo_replicable.scss => Used for /admin/geo/replication/${replicable_type} and /admin/geo/sites/${id}/replication/${replicable_type}
Important: Geo is currently working through a rename from Geo Nodes => Geo Sites. As of right now everything in the UI will reference Sites but the code may still references them as nodes. This follows the Facade Approach to renaming and more can be read here: &5725 (closed)
Screenshots or screen recordings
| Before | After | |
|---|---|---|
| Geo Form | ![]() |
![]() |
| Geo Form (CSS) | ![]() |
![]() |
| Geo Sites | ![]() |
![]() |
| Geo Sites (CSS) | ![]() |
![]() |
| Geo Legacy View | ![]() |
![]() |
| Geo Legacy View (CSS) | ![]() |
![]() |
| Geo Replicable | ![]() |
![]() |
| Geo Replicable (CSS) | ![]() |
![]() |
How to set up and validate locally
- Fetch and checkout this branch
- Setup Geo and access your Primary GDK (How to setup Geo)
- Go to Geo Form (
/admin/geo/sites/new) and ensure CSS is correctly loaded - Go to Geo Sites (
/admin/geo/sites) and ensure CSS is correctly loaded - Go to Geo Legacy Projects View (
/admin/geo/replication/projects) and ensure CSS is correctly loaded - Go to Geo Replicable View (
/admin/geo/sites/1/replication/package_files) and ensure CSS is correctly loaded
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #366610 (closed)















