-
Notifications
You must be signed in to change notification settings - Fork 992
fix visibility dropdown alignment in new ui #13760
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
Conversation
@@ -1576,7 +1576,7 @@ | |||
:<- [:home-items-show-number] | |||
(fn [[search-filter filtered-chats communities view-id home-items-show-number]] | |||
(if (or (= view-id :home) | |||
(and config/new-ui-enabled? (= view-id :chat-stack))) | |||
(and @config/new-ui-enabled? (= view-id :chat-stack))) |
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.
Not related to PR, but a separate small bug reported here
Jenkins Builds
|
90% of end-end tests have passed
Failed tests (9)Click to expandClass TestPairingSyncMultipleDevicesMerged:
Passed tests (78)Click to expandClass TestOnboardingOneDeviceMerged:
Class TestPublicChatMultipleDeviceMerged:
Class TestContactBlockMigrateKeycardMultipleSharedDevices:
Class TestOneToOneChatMultipleSharedDevices:
Class TestSendTxDeviceMerged:
Class TestPublicChatBrowserOneDeviceMerged:
Class TestGroupChatMultipleDeviceMerged:
Class TestEnsStickersMultipleDevicesMerged:
Class TestRestoreOneDeviceMerged:
Class TestWalletManagementDeviceMerged:
Class TestCommandsMultipleDevicesMerged:
Class TestKeycardTxOneDeviceMerged:
|
Hi @Parveshdhull, thanks for PR! |
4a6be21
to
06f2f14
Compare
100% of end-end tests have passed
Passed tests (9)Click to expandClass TestPairingSyncMultipleDevicesMerged:
|
@Parveshdhull thanks for reporting the issue and for the fix! |
fixes #13758
status: ready