Skip to content

Conversation

Hilyxx
Copy link
Contributor

@Hilyxx Hilyxx commented May 30, 2025

Instead of using the cont declaration for avatar size, add an option in the settings to resize the user's avatar in the menu.

Hilyxx added 2 commits May 31, 2025 00:08
Instead of using cont declaration for avatar size, add an option in settings to resize user avatar in the menu.
@Hilyxx Hilyxx changed the title Cinnamon user@applet - Add an option to resize user's avatar in the settings [email protected] - Add an option to resize user's avatar in the settings May 31, 2025
@anaximeno
Copy link
Contributor

anaximeno commented Jun 3, 2025

I'm not sure an option for this is needed directly in settings, at most this should be something defined by the theme (at most) if we're talking about the icon that appears in the popup menu, and the panel one by the panel icon sizes (for colored icons when the user image is used, and for symbolic icons when the symbolic one is used).

The concern here being, bringing unecessary additional complexity in comparison to the actual use potential.

@Hilyxx
Copy link
Contributor Author

Hilyxx commented Jun 3, 2025

"The concern here being, bringing unecessary additional complexity in comparison to the actual use potential."
Of course, I understand your point of view.

The idea is not to make anything more complex, but just to provide an additional option to give users more choice, and there's nothing complex about the option.
As the avatar size (in the menu) is fixed at 64 px, some custom themes might use an incorrect size for the avatar (too big or too small). This would result in an incorrectly sized avatar at high / low screen resolutions.

@mtwebster
Copy link
Member

Overriding the theme isn't really a good case for adding a setting, especially something as minor as this.

@mtwebster mtwebster closed this Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants