Skip to content

fix(avatar): decrease avatar size from 44px to 36px - #12956

Merged
Antreesy merged 3 commits into
mainfrom
feat/noid/shrink-avatars
Aug 15, 2024
Merged

fix(avatar): decrease avatar size from 44px to 36px#12956
Antreesy merged 3 commits into
mainfrom
feat/noid/shrink-avatars

Conversation

@Antreesy

@Antreesy Antreesy commented Aug 13, 2024

Copy link
Copy Markdown
Contributor

☑️ Resolves

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏚️ Before 🏡 After
image image
image image
image image
image image

🏁 Checklist

  • 🌏 Tested with Chrome, Firefox and Safari or should not be risky to browser differences
  • 🖥️ Tested with Desktop client or should not be risky for it
  • 🖌️ Design was reviewed, approved or inspired by the design team
  • ⛑️ Tests are included or not possible
  • 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

Comment thread src/components/LeftSidebar/LeftSidebar.vue Outdated
@Antreesy
Antreesy force-pushed the feat/noid/shrink-avatars branch from 3b4b9eb to c22f684 Compare August 14, 2024 14:18
@Antreesy Antreesy changed the title fix(avatar): decrease avatar size from 44px to 38px fix(avatar): decrease avatar size from 44px to 36px Aug 14, 2024
@Antreesy
Antreesy requested a review from ShGKme August 14, 2024 14:21

@marcoambrosini marcoambrosini left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The position of the status icon also needs to change, likely 2px up and 2px left

@nickvergessen

Copy link
Copy Markdown
Member

/backport to stable30

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
Antreesy force-pushed the feat/noid/shrink-avatars branch from c22f684 to 2bae552 Compare August 15, 2024 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants