-
Notifications
You must be signed in to change notification settings - Fork 43
[C-1351] Finalize mobile-nav-bar-overhaul #2156
Conversation
const IconNotificationn = useMemo( | ||
() => | ||
colorize(IconNotificationBase, { | ||
// Bell.Group 1.Fill 1 |
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.
nice
height: spacing(12), | ||
width: spacing(12), | ||
marginBottom: spacing(3) | ||
height: spacing(12) + 2, |
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.
should we accept partial sizes instead of this?
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.
Yeah will do, it's cause border width, I wonder if there is a clear way to express that
@@ -31,6 +31,7 @@ export const NotificationsScreen = () => { | |||
text={messages.header} | |||
icon={IconNotification} | |||
iconProps={{ height: 28, width: 28 }} | |||
styles={{ icon: { marginLeft: -1 } }} |
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.
how come this is necessary?
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.
Yeah it's weird, the icons have various widths, so this is an attempt to align each icon
[7952185] Auto follow person who referred you upon sign up C-1300 (#2112) nicoback2 [d87645e] Fix connected eth wallets (#2161) Raymond Jacobson [84c13b7] [C-1352] Fix twitter deep link (#2159) Raymond Jacobson [ffdd74a] dont deep link oauth (#2158) nicoback2 [fd75bf0] add use rendering trace util (#2153) nicoback2 [1b18577] [Android] Some optimizations for lineup C-1348 (#2150) nicoback2 [36506fb] [C-1347] Improve navigation performance (#2155) Sebastian Klingler [e6d1db3] [C-1351] Finalize mobile-nav-bar-overhaul (#2156) Dylan Jeffers [780a7c2] [C-1345] redesign primary/secondary screens (#2152) Dylan Jeffers [c6bedda] Update collection tables to allow for default sorting (#2151) Kyle Shanks
Description