Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add hexagon icon shape #4697

Merged
merged 5 commits into from
Aug 23, 2024
Merged

Add hexagon icon shape #4697

merged 5 commits into from
Aug 23, 2024

Conversation

SelfRef
Copy link
Contributor

@SelfRef SelfRef commented Aug 22, 2024

Description

This PR adds a new icon shape - hexagon. Basically this is port of my own PR for NeoApplications/Neo-Launcher#388

From my tests, it works well:

Settings Home List
Screenshot_20240822_192628 Screenshot_20240822_014056 Screenshot_20240822_014119

Type of change

❌ General change (non-breaking change that doesn't fit the below categories like copyediting)
❌ Bug fix (non-breaking change which fixes an issue)
✅ New feature (non-breaking change which adds functionality)
❌ Breaking change (fix or feature that would cause existing functionality to not work as expected)

@SuperDragonXD SuperDragonXD self-requested a review August 22, 2024 06:17
@tgex0
Copy link
Contributor

tgex0 commented Aug 22, 2024

Not a big deal, but currently, all icon shapes are listed in the UI alphabetically (except for System at the very top). Slotting Hexagon between Octagon and One UI would break this trend.

@SelfRef
Copy link
Contributor Author

SelfRef commented Aug 22, 2024

Not a big deal, but currently, all icon shapes are listed in the UI alphabetically (except for System at the very top). Slotting Hexagon between Octagon and One UI would break this trend.

I saw the order is hard-coded, so even if for English it may work like that, for other languages it won't, isn't it?

But yeah, I changed the order so it fits the current scheme.

@MrSluffy MrSluffy merged commit 11ebddb into LawnchairLauncher:14-dev Aug 23, 2024
5 checks passed
nulldrf pushed a commit to nulldrf/lawnchair that referenced this pull request Aug 24, 2024
* Add hexagon icon shape

* Fix formatting

* Change shape order in settings

* Remove polish translation
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.

5 participants