Skip to content

Comments

[FIX] Horizontal scroll on user info tab#12102

Merged
sampaiodiego merged 5 commits intodevelopfrom
fix/user-bar-scroll
Sep 26, 2018
Merged

[FIX] Horizontal scroll on user info tab#12102
sampaiodiego merged 5 commits intodevelopfrom
fix/user-bar-scroll

Conversation

@rssilva
Copy link
Contributor

@rssilva rssilva commented Sep 19, 2018

Closes #11961

I'm fixing errors on modal.css to avoid build breaking

screen shot 2018-09-19 at 9 37 04 am

@rssilva rssilva self-assigned this Sep 19, 2018
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-12102 September 19, 2018 12:45 Inactive
@rssilva rssilva force-pushed the fix/user-bar-scroll branch from 2a3053e to f140534 Compare September 19, 2018 12:46
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-12102 September 19, 2018 12:46 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-12102 September 19, 2018 12:58 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-12102 September 19, 2018 13:07 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-12102 September 19, 2018 13:14 Inactive
@rssilva rssilva added the area: ui Touches the code on client side label Sep 19, 2018
@rssilva rssilva added this to the 0.70.0 milestone Sep 19, 2018
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-12102 September 19, 2018 20:28 Inactive
ggazzo
ggazzo previously approved these changes Sep 19, 2018
@rssilva rssilva force-pushed the fix/user-bar-scroll branch from ef16514 to c2c8580 Compare September 24, 2018 12:58
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-12102 September 24, 2018 12:58 Inactive
&__item {
overflow: hidden;

max-width: 45%;
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this better than adding overflow: hidden in the parent element (.rc-user-info-action)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It isn't! I changed 😄 now! Thank you 🙌

@sampaiodiego sampaiodiego changed the title [FIX] Adding max-width to user action item to avoid horizontal scroll [FIX] Horizontal scroll on user info tab Sep 26, 2018
@sampaiodiego sampaiodiego merged commit 939349d into develop Sep 26, 2018
@sampaiodiego sampaiodiego deleted the fix/user-bar-scroll branch September 26, 2018 18:57
This was referenced Sep 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ui Touches the code on client side

Projects

None yet

Development

Successfully merging this pull request may close these issues.

There is useless scroll in private group members view

5 participants