In D20536 we rename the colorful user avatar to user-man. This patch changes that avatar to the name "Man" to match the new name.
Details
- Reviewers
ngraham afiestas broulik davidedmundson - Group Reviewers
Plasma VDG - Commits
- R128:774c4c628d72: Rename colorful user gallery avatar
Open the avatar gallery.
Diff Detail
- Repository
- R128 User Manager
- Branch
- change-default-user-avatar (branched from master)
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 10886 Build 10904: arc lint + arc unit
Ok, the problem is that now the default user icon doesn't work with a dark theme. I think we should just provide it in breeze-icons, so it is the default user icon that also gets selected when you click Clear Avatar.
This works and makes sense to me, but I'm not as familiar with the user-manager codbease and would appreciate a Plasma review too, in case this might have difficult-to-see consequences.
Ping
For future, the commit message is important.
The person reviewing doesn't have the same context you have when you're making the change, if you can save the reviewer having to dig around user-manager to understand what we're doing it makes everything easier. You won't have me asking potentially stupid questions :D
So from what I can tell:
- UserManager by default uses an icon from breeze
QIcon::fromTheme(QStringLiteral("user-identity"))
It completely ignores anything from the gallery we ship with user-manager /o\
- The "User.png" gallery icon matches the large version of the "user-identity" icon in the breeze theme
The part I don't understand, how does renaming this file change anything?
I changed the title and description, it should be clearer now.
D20536 renames the colorful user avatar to user-man and adds a monochrome icon called user-identity, so that the monochrome icon is the new default user icon. This patch just renames the "User" avatar to "Man" to match the name change in D20536.
So this isn't trying to have any behavioural changes?
Explains why I can't find them :D