Skip to content

Drop the padding left over from the fixed navbar and refresh the screenshots - #344

Merged
lixmal merged 1 commit into
masterfrom
screenshots
Aug 26, 2026
Merged

lixmal merged 1 commit into
masterfrom
screenshots

Conversation

@lixmal

@lixmal lixmal commented Aug 26, 2026

Copy link
Copy Markdown
Owner

The screenshots in the README were taken in March 2024 and show the interface as it was before #326.

Taking them again surfaced a leftover: the body still carried padding-top: 70px from the days of a fixed navbar, and the vault cancelled it for itself, so only the login pages kept it and rendered an empty band above the navbar. Nothing is fixed-position any more except the modal and the mobile panels, so the padding is gone rather than cancelled twice.

  • Remove the body padding and the override the vault needed because of it
  • Replace the login and app screenshots, and add one of the dark theme

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 28 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: df2cd4c3-65a7-4b51-9ed5-26ed552ee98a

📥 Commits

Reviewing files that changed from the base of the PR and between 7775d1c and c2c3b3c.

⛔ Files ignored due to path filters (3)
  • doc/img/app-dark.png is excluded by !**/*.png
  • doc/img/app.png is excluded by !**/*.png
  • doc/img/login.png is excluded by !**/*.png
📒 Files selected for processing (3)
  • README.md
  • js/style/app.css
  • js/style/vault.css

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@lixmal
lixmal merged commit 721ffda into master Aug 26, 2026
10 checks passed
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.

1 participant