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

Minor styling fixes #4802

Merged
merged 2 commits into from
Jun 13, 2020
Merged

Minor styling fixes #4802

merged 2 commits into from
Jun 13, 2020

Conversation

dsoegijono
Copy link
Member

Update README

Need to serve before it's accessible

Desktop table width

The column widths were set on the <td> for icons only, so during search if a section has only non-tfa listed, the width wasn't set. This ensures the table headers width is fixed.

Before
Screen Shot 2020-06-12 at 11 14 52 pm

After
Screen Shot 2020-06-12 at 11 13 59 pm

Site name overlaps exception icon

Before
Screen Shot 2020-06-12 at 10 46 36 pm

After
Screen Shot 2020-06-12 at 10 46 44 pm

Update README
@dsoegijono dsoegijono requested a review from Carlgo11 June 12, 2020 13:19
@dsoegijono dsoegijono added the enhancement Issue/PR contains enhancements to the overall code of the site. label Jun 12, 2020
Add babel-minify to macos instructions
@phallobst
Copy link
Member

I just noticed that the sticky table headings only work in Firefox again, but not in Chrome (+Chromium-based browsers) - see also #4746.

Can you fix that as well? I was unsuccessful.

Copy link
Member

@Carlgo11 Carlgo11 left a comment

Choose a reason for hiding this comment

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

Love it! Is the package-lock.json needed?

@dsoegijono
Copy link
Member Author

@Carlgo11 Not necessary exactly. Do you manually remove it after npm i babel-minify? But since this is a requirement for the site to run, I think we should add package.json and package-lock.json

@phallobst I can try looking into it but maybe in a separate PR?

@phallobst
Copy link
Member

The column width are fixed by this. I'm okay with another PR for the "sticky" bug.

@phallobst phallobst merged commit d7d7dbe into 2factorauth:master Jun 13, 2020
@dsoegijono dsoegijono deleted the style-fixes branch June 13, 2020 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issue/PR contains enhancements to the overall code of the site.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants