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

Request for opinion: Rethink the quickemu overlay #49

Closed
Lukewh opened this issue Jan 6, 2022 · 3 comments
Closed

Request for opinion: Rethink the quickemu overlay #49

Lukewh opened this issue Jan 6, 2022 · 3 comments
Labels
question Further information is requested

Comments

@Lukewh
Copy link
Contributor

Lukewh commented Jan 6, 2022

My original thoughts behind having the distro icon + the quickemu icon overlay was for use in .desktop files and for running VMs.

It's great that quickqui has implemented the icons, but it's my opinion they don't look great in their current format:
image

This is no fault of quickqui, it's an issue with the icons themselves. Used at such a small scale, the quickgui icon becomes noise, and on a dark background, the white circle becomes jarring.

Until there is a way to use these icons for running VMs and .desktop files, I propose we also provide the distro icons without the quickemu icon overlay.

Edit This may help with situations as described here

@Lukewh Lukewh added the question Further information is requested label Jan 6, 2022
@daPhipz
Copy link
Member

daPhipz commented Jan 6, 2022

Used at such a small scale, the quickgui icon becomes noise, and on a dark background, the white circle becomes jarring.

Yeah, I think so too. Additionally, the purpose of the quickgui overlay is to convey the information that this desktop launcher has something to do with Quickgui. Since the icons mentioned above are inside the program itself, this extra info is not necessary.

@philclifford
Copy link
Contributor

I think you're right: for quickgui menu use the overlay is too tiny and as daPhipz said somewhat unnecessary .

@Lukewh
Copy link
Contributor Author

Lukewh commented Nov 21, 2022

I have included the base icons in the build - I believe this can now be closed <3

@Lukewh Lukewh closed this as completed Nov 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants