-
Notifications
You must be signed in to change notification settings - Fork 984
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: quo2 floating-shell-button click through #16030
Conversation
Jenkins Builds
|
@@ -114,7 +114,7 @@ | |||
|
|||
(defn calc-shell-neg-y | |||
[insets maximized? extra-height] | |||
(let [padding 6 | |||
(let [padding 12 |
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.
Why this change?
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.
Previously the floating-shell-button
had a padding of 5px
which I removed in this PR. To compensate for this modification, I have adjusted the padding here.
88% of end-end tests have passed
Failed tests (4)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
Passed tests (29)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityCenterContactRequestMultipleDevicePR:
|
Hi @ajayesivan thank you for your work. PR is ready to be merged |
a1beae7
to
51885be
Compare
fix #15812
status: ready