Skip to content

Night mode fixes #1163

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

Merged
merged 5 commits into from
Feb 28, 2023
Merged

Conversation

peterbecich
Copy link
Member

@peterbecich peterbecich commented Jan 7, 2023

Night mode fixes

@peterbecich
Copy link
Member Author

looking at haskell/haddock#790 (comment)

@peterbecich
Copy link
Member Author

@gbaz
Copy link
Contributor

gbaz commented Jan 7, 2023

Right. just build a doc tarball with quickjump enabled, then upload it. No need to run a docbuilder instance yourself etc.

@peterbecich
Copy link
Member Author

Big improvement, @gbaz please review

Screenshot 2023-01-06 at 20-49-56 steeloverseer

Screenshot 2023-01-06 at 20-50-27 steeloverseer

@peterbecich peterbecich marked this pull request as ready for review January 7, 2023 04:51
@peterbecich peterbecich changed the title [WIP] Night mode quickjump fix Night mode quickjump fix Jan 7, 2023
@peterbecich peterbecich requested a review from gbaz January 7, 2023 04:53
@gbaz
Copy link
Contributor

gbaz commented Jan 7, 2023

Also testing this on prod. Looks much better! Still not perfect, imho -- the light blue of the shortcut keys isn't great, and the highlighting when mousing-over a search result washes out the light text. However, its certainly much more usable than before. Now that we have this and the other set of changes on prod, I think the immediate urgency has passed. I'd certainly like to see some future work to make this really nice, based on your own (good) taste and user feedback, but there's no rush, and its around midnight my time, so I certainly don't intend to think about this for the next little bit :-) Cheers!

@peterbecich
Copy link
Member Author

@gbaz , agreed, I will improve it further later. Thank you for your critical assistance and project leadership! Have a great weekend

@peterbecich peterbecich changed the title Night mode quickjump fix Night mode fixes Jan 28, 2023
@peterbecich
Copy link
Member Author

peterbecich commented Jan 29, 2023

Closes: #1167

I could not reproduce this issue with the Firefox dev tools, but probably fixed by a9ca79c

Could be further improved: https://stackoverflow.com/questions/7187569/difference-in-applying-css-to-html-body-and-the-universal-selector

Screen Shot 2023-01-28 at 16 21 22

Screenshot 2023-01-28 at 16-23-10 Introduction Hackage

Screenshot 2023-01-28 at 16-22-00 Introduction Hackage

@peterbecich
Copy link
Member Author

@gbaz , can we merge this now to close #1167 and make the other fixes #1163 (comment) later?

@andreasabel
Copy link
Member

@peterbecich : Are these commits self-contained and should be preserved, or should we squash&merge them?

@peterbecich
Copy link
Member Author

@andreasabel , squash would be good

@peterbecich peterbecich merged commit 7168298 into haskell:master Feb 28, 2023
@peterbecich peterbecich deleted the night-mode-quickjump-fix branch February 28, 2023 07:11
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.

3 participants