Fix: consistent heading structure across all admin screens#365
Merged
rami-elementor merged 5 commits intocore-betafrom May 1, 2026
Merged
Fix: consistent heading structure across all admin screens#365rami-elementor merged 5 commits intocore-betafrom
rami-elementor merged 5 commits intocore-betafrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds the shared top toolbar to the Import screen and adjusts the Import/Migrate UI structure/styles to fit the updated page layout and heading hierarchy.
Changes:
- Render the shared
Toolbaron the Import screen and adjust the Import page heading/layout containers. - Normalize heading levels across Import/Migrate cards and tweak related SCSS (including max-width handling).
- Extend toolbar nav-link metadata (
pageSlug/subpage) to improve active-link detection.
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/js/components/ImportMenu/UploadForm/UploadForm.tsx | Removes nested .wrap/container wrappers so Upload step fits within the Import page layout. |
| src/js/components/ImportMenu/UploadForm/SelectSnippets/SelectSnippets.tsx | Adjusts the “Available snippets” heading level in the selection step. |
| src/js/components/ImportMenu/UploadForm/SelectSnippets/ImportResultDisplay.tsx | Updates result icon accessibility and adjusts result heading level. |
| src/js/components/ImportMenu/UploadForm/SelectFiles/SelectFiles.tsx | Shifts card heading level to match the new Import page hierarchy. |
| src/js/components/ImportMenu/UploadForm/SelectFiles/SelectedFilesList.tsx | Shifts selected-files heading level to match the new Import page hierarchy. |
| src/js/components/ImportMenu/UploadForm/SelectFiles/DuplicateActionSelector.tsx | Shifts duplicate-action card heading level to match the new Import page hierarchy. |
| src/js/components/ImportMenu/MigrateForm/SimpleSnippetTable.tsx | Shifts migrate table card heading level to align with updated hierarchy. |
| src/js/components/ImportMenu/MigrateForm/MigrateForm.tsx | Removes extra container wrapper and adjusts status/no-results headings. |
| src/js/components/ImportMenu/MigrateForm/ImportOptions.tsx | Shifts “Import options” heading level for hierarchy consistency. |
| src/js/components/ImportMenu/MigrateForm/ImporterSelector.tsx | Shifts “Select plugin” heading level for hierarchy consistency. |
| src/js/components/ImportMenu/ImportMenu.tsx | Adds Toolbar and updates Import page title/structure. |
| src/js/components/EditMenu/SnippetForm/SnippetForm.tsx | Removes .wrap wrapper around the edit form content. |
| src/js/components/common/Toolbar.tsx | Adds pageSlug/subpage metadata and refactors active-link detection. |
| src/css/import/_upload.scss | Updates selectors to match revised heading levels and removes width container styling. |
| src/css/import/_page.scss | Updates Import page header and section sizing/layout rules. |
| src/css/import/_migrate.scss | Updates selectors to match revised heading levels and removes width container styling. |
| src/css/import/_card.scss | Ensures card sizing includes borders via box-sizing: border-box. |
| src/css/import.scss | Imports toolbar styles into the import stylesheet bundle. |
rami-elementor
changed the title
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.
More updated after #362 :
<div>elements.wrapclasses.wrapclasses to upper<div>