New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(toolbar): added item width control #3994
feat(toolbar): added item width control #3994
Conversation
Preview: https://patternfly-pr-3994.surge.sh A11y report: https://patternfly-pr-3994-coverage.surge.sh CSS Size Report
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks OK to me, but I'll defer to others for approval.
|
||
| Class | Applied to | Outcome | | ||
| -- | -- | -- | | ||
| `--pf-c-toolbar__item--Width{-on-[breakpoint]}: {width}` | `.pf-c-toolbar__item` | Modifies the width value of a `toolbar item`. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| `--pf-c-toolbar__item--Width{-on-[breakpoint]}: {width}` | `.pf-c-toolbar__item` | Modifies the width value of a `toolbar item`. | | |
| `--pf-c-toolbar__item--Width{-on-[breakpoint]}: {width}` | `.pf-c-toolbar__item` | Modifies the width value of a toolbar item. | |
@@ -133,3 +148,4 @@ When using the `.pf-c-input-group` always ensure labels are used outside the inp | |||
| `.pf-c-input-group` | `<div>` | Initiates the input group. **Required** | | |||
| `.pf-c-input-group__text` | `<span>` | Initiates the input group text. This can be used to show text, radio, icons, or check boxes. | | |||
| `.pf-m-plain` | `.pf-c-input-group__text` | Removes the border from the text element. | | |||
| `--pf-c-input-group--Width{-on-[breakpoint]}: {width}` | `.pf-c-input-group` | Modifies the width value of an `input group`. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| `--pf-c-input-group--Width{-on-[breakpoint]}: {width}` | `.pf-c-input-group` | Modifies the width value of an `input group`. | | |
| `--pf-c-input-group--Width{-on-[breakpoint]}: {width}` | `.pf-c-input-group` | Modifies the width value of an input group. | |
@@ -116,6 +116,21 @@ cssPrefix: pf-c-input-group | |||
{{/input-group}} | |||
``` | |||
|
|||
### Width control |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm wondering if we need this for an input group, or if we could just enable it on a toolbar item? Was there a reason we need it on an input group, too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My reasoning was that the same issue would occur outside of a toolbar
. But we can limit this PR to toolbar
59fc44c
to
e3dcd88
Compare
@mcoker updated |
7693984
to
928de2c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
L🏆TM!!
🎉 This PR is included in version 4.99.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
closes #3894
closes #3449
Added width control to and
toolbar item
.