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

Jump to bottom

Add missing shortcut icons #521

Merged
merged 2 commits into from Nov 26, 2022
Merged

Add missing shortcut icons #521

merged 2 commits into from Nov 26, 2022

Conversation

Scrumplex
Copy link
Member

soda

@Scrumplex Scrumplex added the enhancement New feature or request label Nov 25, 2022
@Scrumplex Scrumplex added this to the 6.0 milestone Nov 25, 2022
@DioEgizio
Copy link
Member

DioEgizio commented Nov 25, 2022 •
edited

The breeze ones are missing

@Scrumplex
fix: add missing shortcut icons
cbf4159 
Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net>
@DioEgizio
Copy link
Member

DioEgizio commented Nov 26, 2022 •
edited

For consistency might make more sense to use https://github.com/KDE/breeze-icons/blob/master/icons-dark/actions/16/link.svg / https://github.com/KDE/breeze-icons/blob/master/icons/actions/16/link.svg instead of the colored one on breeze, otherwise lgtm

@DioEgizio
fix: use monochrome icon for breeze
c90f2c3 
Signed-off-by: DioEgizio <83089242+DioEgizio@users.noreply.github.com>
@DioEgizio DioEgizio merged commit cc92e1b into PrismLauncher:develop Nov 26, 2022
@DioEgizio DioEgizio mentioned this pull request Nov 27, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Leo40Git Leo40Git Leo40Git approved these changes

@flowln flowln flowln approved these changes

@DioEgizio DioEgizio DioEgizio approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
6.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Scrumplex @DioEgizio @Leo40Git @flowln