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

Fixed dark mode issues #521

Merged
merged 7 commits into from Jul 13, 2021
Merged

Fixed dark mode issues #521

merged 7 commits into from Jul 13, 2021

Conversation

jt15s
Copy link
Contributor

@jt15s jt15s commented Jun 22, 2021

Fixed issues in dark mode on the installation guide and the Get Haiku! page.

Made text colour black for the Installation Guide steps so the text is visible in dark mode.
@jt15s jt15s marked this pull request as draft June 22, 2021 22:58
Fixed issues in dark mode on the installation guide and the Get Haiku! page.
@jt15s jt15s changed the title Update shijin4.css Fixed dark mode issues Jun 23, 2021
@jt15s jt15s marked this pull request as ready for review June 23, 2021 04:07
Update copyright date
Fixed port status page dark mode display issues, plus added table hover effects.
@jt15s
Copy link
Contributor Author

jt15s commented Jun 24, 2021

Would be really appreciated if someone reviewed this!

Copy link
Sponsor Collaborator

@humdingerb humdingerb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't comment on the other changes, just spotted the key->button transition and wondered why...

content/get-haiku/installation-guide.html Outdated Show resolved Hide resolved
Restore key spans as suggested by @humdingerb
@jt15s
Copy link
Contributor Author

jt15s commented Jun 26, 2021

@humdingerb I've just restored the key spans.

@humdingerb
Copy link
Sponsor Collaborator

OK. I still have no clue about dark mode, so I'm outta here...

Copy link
Contributor

@leavengood leavengood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So how does dark mode work? Is it a browser feature? Can you say what browser you are testing with? Because I would want to make sure this CSS changes don't affect the normal "light mode". Also this needs rebasing after the recent Bootstrap 4 upgrade

content/get-haiku/installation-guide.html Outdated Show resolved Hide resolved
@pulkomandy
Copy link
Member

Yes, the browser has a way to say "I'm using dark mode" and the CSS can query that and use different colors depending on it.
You can test this in recent versions of WebPositive if you have set your Haiku system colors to a dark theme.

In other browsers it's probably an option somewhere in preferences.

jt15s added a commit to jt15s/website that referenced this pull request Jul 13, 2021
Add link classes to ensure table is visible (though this is dependent on haiku#521 passing) and reverted change of graphic to Beta2 download button to prevent any confusion once this is merged.
Co-authored-by: Ryan Leavengood <leavengood@gmail.com>
@jt15s
Copy link
Contributor Author

jt15s commented Jul 13, 2021

Looks like this has a conflict with the shijin4.css file in master - could one of the repo maintainers fix this please and merge if there are no remaining issues? Thanks in advance!

@leavengood leavengood merged commit 7bc1b83 into haiku:master Jul 13, 2021
@jt15s jt15s deleted the dark-mode-fixes branch July 13, 2021 01:55
leavengood added a commit that referenced this pull request Jul 13, 2021
* Added tags to blog posts and added WIP pages for Beta 3 release details

Added tags to some of the blog posts related to the Haiku Promotion Team, and also added WIP release pages for Beta 3 (based on the ones for Beta 2) - these have been marked as "draft=true" so Hugo will not publish these. Feel free to add any info/corrections!

* Update release-notes.md

Made improvements as suggested by @nielx

* Updated Getting Involved system page and ports status page

Updated the Getting Involved system page to remove outdated parts, and added X512's RISC-V port to the ports status page.

* Update GET_HAIKU.PNG

Updated download button graphic for Beta3.

* Update social-media.md

Resolved a merge conflict.

* Update port_status.html

Fixed wrong target names. Thanks @X547 for pointing that out!

* Update _index.md

Added warning/info boxes.

* Update release-notes.md

Added to-do items plus info boxes.

* Update _index.md

Fixed formatting typos with info boxes.

* Update README.md

Added info on running scripts on Windows.

* Update Beta3 pages

Add link classes to ensure table is visible (though this is dependent on #521 passing) and reverted change of graphic to Beta2 download button to prevent any confusion once this is merged.

Co-authored-by: Ryan Leavengood <leavengood@gmail.com>
kallisti5 pushed a commit to kallisti5/website that referenced this pull request Jan 2, 2022
* Update shijin4.css

Update copyright date
Made text colour black for the Installation Guide steps so the text is visible in dark mode.

* Fixed dark mode issues

Fixed issues in dark mode on the installation guide and the Get Haiku! page.

* Fix port status page dark mode issues

Fixed port status page dark mode display issues, plus added table hover effects.

* Update installation-guide.html

Restore key spans as suggested by @humdingerb

* Update content/get-haiku/installation-guide.html
kallisti5 pushed a commit to kallisti5/website that referenced this pull request Jan 2, 2022
* Added tags to blog posts and added WIP pages for Beta 3 release details

Added tags to some of the blog posts related to the Haiku Promotion Team, and also added WIP release pages for Beta 3 (based on the ones for Beta 2) - these have been marked as "draft=true" so Hugo will not publish these. Feel free to add any info/corrections!

* Update release-notes.md

Made improvements as suggested by @nielx

* Updated Getting Involved system page and ports status page

Updated the Getting Involved system page to remove outdated parts, and added X512's RISC-V port to the ports status page.

* Update GET_HAIKU.PNG

Updated download button graphic for Beta3.

* Update social-media.md

Resolved a merge conflict.

* Update port_status.html

Fixed wrong target names. Thanks @X547 for pointing that out!

* Update _index.md

Added warning/info boxes.

* Update release-notes.md

Added to-do items plus info boxes.

* Update _index.md

Fixed formatting typos with info boxes.

* Update README.md

Added info on running scripts on Windows.

* Update Beta3 pages

Add link classes to ensure table is visible (though this is dependent on haiku#521 passing) and reverted change of graphic to Beta2 download button to prevent any confusion once this is merged.

Co-authored-by: Ryan Leavengood <leavengood@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants