Skip to content

Conversation

@rowantran
Copy link
Contributor

Remove the white-space property from .btn, which prevents accordion titles from overflowing when they are wider than the viewport.

Closes #26954

@rowantran
Copy link
Contributor Author

After removing nowrap:
accordion-wrap

@XhmikosR XhmikosR requested a review from andresgalante August 12, 2018 21:41
Copy link
Collaborator

@andresgalante andresgalante left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @rowantran

@mdo mdo merged commit 24bdadf into twbs:v4-dev Aug 19, 2018
@mdo mdo mentioned this pull request Aug 19, 2018
@rowantran rowantran deleted the btn-remove-nowrap branch August 19, 2018 23:04
pparidans added a commit to odoo-dev/odoo that referenced this pull request Apr 8, 2019
Buttons "Send message / Log note / Activity" are on two lines instead of
one. In addition there is a 1px glitch on the attachment icon.

Also long attachment name on tiny screens (i.e. iPhone 5) breaks the
composer layout: horizontal scrollbar appears.

The two-line buttons effect is related to a change (fix) in Bootstrap
4.2 which removes the white-space: nowrap on .btn. It was originally
introduced in Odoo through odoo#29949.

Reference: Original Bootstrap PR twbs/bootstrap#27010

Task ID: 1946573
robodoo pushed a commit to odoo/odoo that referenced this pull request Apr 8, 2019
Buttons "Send message / Log note / Activity" are on two lines instead of
one. In addition there is a 1px glitch on the attachment icon.

Also long attachment name on tiny screens (i.e. iPhone 5) breaks the
composer layout: horizontal scrollbar appears.

The two-line buttons effect is related to a change (fix) in Bootstrap
4.2 which removes the white-space: nowrap on .btn. It was originally
introduced in Odoo through #29949.

Reference: Original Bootstrap PR twbs/bootstrap#27010

Task ID: 1946573

closes #31939

Signed-off-by: Adrien Dieudonné (adr) <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants