Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Dec 16, 2019

Fixes element-hq/element-web#8456
Fixes element-hq/element-web#11287

Based on new design:
image
image

Result:
image
image
image

@t3chguy
Copy link
Member Author

t3chguy commented Dec 16, 2019

Blocked on #3732 otherwise it not possible to interact with the ImageView atop the Settings

@t3chguy
Copy link
Member Author

t3chguy commented Dec 16, 2019

Needs design for no-avatar state in Room Profile Settings

image

@t3chguy t3chguy requested a review from a team December 16, 2019 16:20
…ithub.com/matrix-org/matrix-react-sdk into t3chguy/remove_avatar_2.0 � Conflicts: �	res/css/views/settings/_ProfileSettings.scss
@dbkr dbkr requested review from dbkr and removed request for a team December 16, 2019 16:51

// show a different AvatarSetting placeholder for RoomProfileSettings which is basically a clone of ProfileSettings
.mx_RoomSettingsDialog .mx_AvatarSetting_avatar .mx_AvatarSetting_avatarPlaceholder::before {
mask: url("$(res)/img/feather-customised/image.svg");
Copy link
Member

Choose a reason for hiding this comment

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

avatar_placeholder.svg or something maybe?

Copy link
Member Author

Choose a reason for hiding this comment

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

Nooooooo

Copy link
Member

Choose a reason for hiding this comment

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

Oh I see, it really is an image

@t3chguy t3chguy merged commit dd216d1 into develop Dec 16, 2019
@t3chguy t3chguy deleted the t3chguy/remove_avatar_2.0 branch May 25, 2020 18:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants