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

Fix footer/menu popup colours #33

Merged

Conversation

misstickles
Copy link
Contributor

When using a dark theme colour (coolblack; spacegrey):

  • the home-page main popup menu (eg on a phone) has a dark grey background and black text
  • the footer background is not applied, giving white text on a white background

This fix aims to solve these two issues.

CSS is NOT my thing, and I don't know your intention using unset and transparent - but this change at least appears correct.

Thanks for the theme - it's sooo good!

@jeffreytse jeffreytse self-assigned this May 17, 2021
@jeffreytse jeffreytse added the bug Something isn't working label May 17, 2021
Copy link
Owner

@jeffreytse jeffreytse left a comment

Choose a reason for hiding this comment

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

Thanks for the corrections.

  1. For the menu popup colour, it absolutely needs to be assigned the right colour to text.

  2. But for the footer colour, the purpose of footer background should be transparent when the body content is less, as below image.

image

When the body content is less, the footer background colour should be transparent.
@jeffreytse jeffreytse merged commit fa28475 into jeffreytse:master May 17, 2021
shahbaz22 pushed a commit to shahbaz22/resonance-site that referenced this pull request Nov 17, 2024
* Fix footer/menu popup colours

* fix: correct the footer background colour

When the body content is less, the footer background colour should be transparent.

Co-authored-by: JT <[email protected]>
jnooree pushed a commit to jnooree/jnooree.github.io that referenced this pull request Feb 17, 2025
Bumps [actions/deploy-pages](https://github.com/actions/deploy-pages) from 3 to 4.
- [Release notes](https://github.com/actions/deploy-pages/releases)
- [Commits](actions/deploy-pages@v3...v4)

[skip ci]

---
updated-dependencies:
- dependency-name: actions/deploy-pages
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants