Skip to content
This repository has been archived by the owner on Feb 25, 2024. It is now read-only.

refactor: moved yaruvariant methods inside the enum #403

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

TheLiux
Copy link
Contributor

@TheLiux TheLiux commented Nov 14, 2023

Description
This pull request addresses code organization concerns by relocating the methods of YaruVariant within the enum itself. This refactor takes advantage of the enhanced enums introduced in Dart 2.17. The primary goal is to centralize YaruVariant related code inside the enum itself.

This was one of the edit proposed in this pull request (#398), which i closed and splitted in more pull request as requested

Here is a brief video to show that the variant switch correctly
https://github.com/ubuntu/yaru.dart/assets/62995808/ee095f7d-10f7-4414-9396-1c907b676d00

@TheLiux
Copy link
Contributor Author

TheLiux commented Nov 14, 2023

@Feichtmeier here is one of the two pull request 😃
If you need a test with a GNOME Linux OS and an application I will provide soon as possible

@Feichtmeier
Copy link
Member

@Feichtmeier here is one of the two pull request 😃
If you need a test with a GNOME Linux OS and an application I will provide soon as possible

Thanks for the pr

Yes we need to check if it works from gnome control center because this is what the YaruTheme is for

@TheLiux
Copy link
Contributor Author

TheLiux commented Nov 14, 2023

@Feichtmeier here is one of the two pull request 😃
If you need a test with a GNOME Linux OS and an application I will provide soon as possible

Thanks for the pr

Yes we need to check if it works from gnome control center because this is what the YaruTheme is for

Ok, I have atm my M1 Macbook, when i go back home i will test with my Pc with Pop OS, okay?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants