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

overhaul breeze dark #42

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

DioEgizio
Copy link
Member

@DioEgizio DioEgizio commented Mar 22, 2024

  • created a new breeze-ish-dark theme based on the old breeze-dark and qbreeze with a slightly darker colour scheme
  • rename Breeze-Dark to MultiMC-Dark

@TheKodeToad
Copy link
Member

I assume the reason it's called this was a misunderstanding?
I don't think it has anything in common with breeze...

@DioEgizio DioEgizio marked this pull request as draft March 22, 2024 11:50
@DioEgizio
Copy link
Member Author

I assume the reason it's called this was a misunderstanding? I don't think it has anything in common with breeze...

you mean this or the old one? for this I changed the color scheme to make it a bit darker, this is how it would look without it

image

but i think it looks worse. Maybe I should change name

@DioEgizio DioEgizio marked this pull request as ready for review March 22, 2024 11:56
@DioEgizio
Copy link
Member Author

I renamed it to Breeze-ish-dark btw

now breeze dark actually looks like Breeze thanks to QBreeze, the old theme is still available as "MultiMC-dark"
@DioEgizio DioEgizio force-pushed the overhaul-breeze-dark branch from 4e19739 to af2567f Compare April 1, 2024 18:06
@TheKodeToad
Copy link
Member

Looks pretty good visually but there are a few issues

Screenshots are invisible:
image

Launch button is weird when hovering:
image
(also tool buttons in general seem to shift 1px to the right when hovering)

Checkbox states only appear when selected or hovered:

Screencast.from.2024-10-21.11-52-01.mp4

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.

2 participants