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

Update: Contrast Edits #66

Closed
wants to merge 6 commits into from
Closed

Conversation

RuKira
Copy link

@RuKira RuKira commented Nov 27, 2024

  • Updated listing.blade.php
  • Updated show.blade.php

To give a summary of what is changed, I basically updated some bg-white to be bg-gray-50 and darken some other text/colors. I made some blues more blue, and such. It's very minor changes, and should not really be noticeable except for contrasting each other.

Default:

Light

image
image

Dark

image
image

Updated:

Light

image
image

Dark

image
image

Keep this PR open so I can make edits and still verify things.
On the off chance these are not dark/light enough edits can be made to fix that.
Also I can edit other views to follow the same scheme, in this same PR.

refringe and others added 5 commits October 3, 2024 16:11
Removes mods that are no longer on the hub.
- Updated listing.blade.php
- Updated show.blade.php

To give a summary of what is changed, I basically updated some bg-white to be bg-gray-50 and darken some other text/colors. I made some blues more blue, and such. It's very minor changes, and should not really be noticeable except for contrasting each other.
@RuKira
Copy link
Author

RuKira commented Nov 27, 2024

Now to go fix my GPG key.
Fixed.

@RuKira
Copy link
Author

RuKira commented Nov 28, 2024

Ahah! Git auto merged, I need to remake this PR how fun.
That is my fault, was trying to test something

@refringe
Copy link
Contributor

These look good! When you think it's in a good place mark it for review. :)

@RuKira
Copy link
Author

RuKira commented Nov 28, 2024

@refringe the question here is would like me to just do these two for now and come back to others later? or more...

If you are ok with these two for now, I'll mark it now.

I'll mark it, I can always just make another PR for any other edits tbh.

You might have to cherry pick also, since I accidently merged main into this.. :/

@RuKira RuKira marked this pull request as ready for review November 28, 2024 16:15
@refringe refringe added the enhancement New feature or request label Nov 28, 2024
@refringe
Copy link
Contributor

Related to #30

@RuKira Before I merge, can you pull the latest changes to develop into your fork, rebase your fork branch contrast-changes on those latest changes, and then force-push your branch back up?

@RuKira RuKira closed this Nov 29, 2024
@RuKira RuKira deleted the contrast-changes branch November 29, 2024 00:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants