Skip to content

Set search input size to 16px #119 #120

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 1 commit into from
May 3, 2020
Merged

Set search input size to 16px #119 #120

merged 1 commit into from
May 3, 2020

Conversation

dpanov
Copy link
Contributor

@dpanov dpanov commented May 3, 2020

Fixes #119 .

Btw, the docs.css file looks like it could use some refactoring.
Here's some stuff I was thinking of doing:

  • Change the file to be a SCSS file
  • Nest media queries inside CSS selectors (e.g. the way it's done in src/theme/BackerSection/backers.module.scss)
  • Group together the selectors - e.g. we are referencing .navbar__search-input in three different places. It will be easier to maintain if the CSS for the selector is in one place.
  • Lower the specificity of some selectors - this could be tricky, because 1. Infima has some unnecessary heavy selectors and 2. the CSS bundling is weird and puts the custom CSS (docs.css) somewhere in the middle of the output file and its selectors are being overwritten by other CSS.

If it's OK I can open a separate PR for these?

Set search input font family to "Poppins";
@phated
Copy link
Member

phated commented May 3, 2020

@dpanov Thank you! I love all of your ideas. Happy to review any PRs that you send for those.

We might also loop in @yangshun and @lex111 if we need anything changed in Docusaurus or Infima (like the specificity).

@phated phated merged commit ccaabb4 into gulpjs:source May 3, 2020
@lex111
Copy link

lex111 commented May 3, 2020

In fact, a few months ago, in the Infima repository (it is private), I opened a PR that removes redundant specificity, but so far that PR has not been reviewed, so this is a good reminder for @yangshun.

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.

Ensure navbar searchbox has font-size 16px or greater
3 participants