Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix duplicate vertical scroll bar when hide status bar #11

Merged
merged 2 commits into from
Apr 24, 2023
Merged

Fix duplicate vertical scroll bar when hide status bar #11

merged 2 commits into from
Apr 24, 2023

Conversation

nqhhdev
Copy link
Member

@nqhhdev nqhhdev commented Apr 20, 2023 •
edited
Loading

  • Issue :
  • When replying to a long mail I have 2 scroll bar on the side of the composer.
    230274312-95492038-b29a-4c03-bba9-f1fed1909a4f
  • When enabling code view
Screen.Recording.2023-04-21.at.23.49.39.mov
3.mov
1.mov
  • Code View
Screen.Recording.2023-04-21.at.23.53.14.mov

@nqhhdev nqhhdev force-pushed the hotfix/duplicate_vertical_scroll_bar branch from ea9a897 to 91dcba5 Compare April 20, 2023 18:32
@nqhhdev nqhhdev mentioned this pull request Apr 21, 2023
Merged
@nqhhdev
Fix duplicate vertical scroll bar when hide statusbar
84fa2cc 
Rebuild HTML widget when change height
@nqhhdev nqhhdev force-pushed the hotfix/duplicate_vertical_scroll_bar branch from 91dcba5 to 84fa2cc Compare April 21, 2023 05:11
dab246
dab246 reviewed Apr 21, 2023
View reviewed changes
Copy link
Member

@dab246 dab246 left a comment

Choose a reason for hiding this comment

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

Push only the things that need to be changed. IMO, example doesn't need to be pushed.

@nqhhdev nqhhdev force-pushed the hotfix/duplicate_vertical_scroll_bar branch from 0a7d291 to c1f18f4 Compare April 24, 2023 02:36
@dab246 dab246 merged commit dc173ea into email_supported Apr 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dab246 dab246 dab246 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nqhhdev @dab246