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
fix: make compact tokenizer inner width: auto and make cozy mode horizontally scrollable #775
Conversation
Deploy preview for fundamental-styles ready! Built with commit 447466c |
146e410
to
59edd97
Compare
Hello @mikerodonnell89 is it done by purpose? |
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.
@@ -280,3 +285,75 @@ To show focus on whole element, it's needed to add `.is-focus` into container. | |||
</div> | |||
{% endcapture %} | |||
{% include display-component.html component=compact-tokenizer %} | |||
|
|||
## Tokenizer - Scrollable | |||
The Compact tokenizer is for large screens. |
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.
This is the cozy tokenizer. And the user won't have tab keys in cozy mode (phone/tablet)
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.
In my my approach, there is added --scrollable
, which can be added in cozy and compact mode.
@mikerodonnell89 |
src/tokenizer.scss
Outdated
@@ -38,7 +44,7 @@ $block: #{$fd-namespace}-tokenizer; | |||
&__input { | |||
border: none !important; | |||
min-width: 3rem !important; | |||
padding: 0 $fd-tokenizer-spacing 0 0 !important; | |||
padding: 0 !important; |
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.
Did we want to keep with the important!
pattern here or switch to the new pattern we did with shellbar buttons?
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.
Now it's handled like on shellbar, thanks for pointing it out :)
Related Issue
Closes SAP/fundamental-styles# #774