-
Notifications
You must be signed in to change notification settings - Fork 441
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
feat(conversations): compact view #14099
Conversation
3e4057c
to
015902a
Compare
c6b43b3
to
64a210a
Compare
015902a
to
3f35371
Compare
Signed-off-by: Dorra Jaouad <dorra.jaoued7@gmail.com>
64a210a
to
2647b89
Compare
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.
Merge-able
src/components/LeftSidebar/ConversationsList/ConversationsListVirtual.vue
Show resolved
Hide resolved
@@ -300,6 +300,7 @@ export const BOT = { | |||
|
|||
export const AVATAR = { | |||
SIZE: { | |||
COMPACT: 20, |
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.
24, as the height of .list-item-content
?
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.
.list-item-content
is 20. Yet, I can see your point that the icon looks smaller than the icon in the settings left sidebar.
☑️ Resolves
REMARK: avatar size was explicitly chosen 20px to match the appearance of settings LS, but we can make it EXTRA_SMALL (22px)
🖌️ UI Checklist
🖼️ Screenshots / Screencasts
🚧 Tasks
🏁 Checklist