docs(next-prev): adding <uxdot-pathfinder> component and adding next/prev links to Getting Started - #3100
Open
markcaron wants to merge 10 commits into
Open
docs(next-prev): adding <uxdot-pathfinder> component and adding next/prev links to Getting Started#3100markcaron wants to merge 10 commits into
<uxdot-pathfinder> component and adding next/prev links to Getting Started#3100markcaron wants to merge 10 commits into
Conversation
…ev links to getting started
✅ Deploy Preview for red-hat-design-system ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Contributor
|
Size Change: +1.51 kB (+0.47%) Total Size: 323 kB 📦 View Changed
ℹ️ View Unchanged
|
Contributor
Documentation HealthNo relevant changes in this PR. |
markcaron
marked this pull request as ready for review
July 15, 2026 12:34
zeroedin
reviewed
Jul 15, 2026
zeroedin
approved these changes
Jul 15, 2026
zeroedin
approved these changes
Jul 15, 2026
zeroedin
left a comment
zeroedin
left a comment
Collaborator
There was a problem hiding this comment.
Links Guiding Through Material
adamjohnson
requested changes
Jul 15, 2026
adamjohnson
left a comment
adamjohnson
left a comment
Collaborator
There was a problem hiding this comment.
Definitely look into the reflow issue.
Also, pending further instruction from design, IMO instituting some min-inline-size and max-inline-size's would probably tighten up the design.
Collaborator
There was a problem hiding this comment.
FYI if you scrunch your browser down to 320px, this component causes horizontal scrollbars. This violates WCAG 1.4.10 Reflow (AA).
I would recommend hiding the rh-icon arrows at narrow browser widths. Designers may have input as to whether these links should stack at small viewport widths.
coreyvickery
requested changes
Jul 15, 2026
Collaborator
There was a problem hiding this comment.
@markcaron These changes should solve everything. cc @adamjohnson
- Change title and page name text sizes to
--rh-font-size-body-text-sm(14px) - Change left and right padding to
--rh-space-lg(16px) - Remove
min-align-sizefrom CSS - Align
Previousto left - Align
Nextto right
Screenshot at 320px
Collaborator
|
@markcaron Not sure if I'm seeing the changes I requested in regards to text size and padding. |
Collaborator
Author
|
@coreyvickery my fault. Try again. |
coreyvickery
requested changes
Jul 28, 2026
coreyvickery
left a comment
coreyvickery
left a comment
Collaborator
There was a problem hiding this comment.
@markcaron Let's remove the 64px padding on top and we're good to go.
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
4 participants
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.
What I did
<uxdot-pathfinder>Testing Instructions