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

v3.chrome-fixes #6084

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

v3.chrome-fixes #6084

wants to merge 5 commits into from

Conversation

PavelLaptev
Copy link
Contributor

Pull Request Description

Title: v3.chrome-fixes

Summary:
This pull request introduces several enhancements and fixes at improving the styling and maintainability of the codebase, particularly Chrome compatibility.

Changes Made:

  1. Refactor CSS Rules:

    • Removed the use of !important and implemented the @layer at-rule to enhance style management and avoid conflicts. This change promotes better control over CSS specificity.
  2. Code Readability Improvements:

    • Replaced class usages with variables to increase code readability and prevent overrides of usual CSS properties. This change aims to streamline the styling process and make future updates easier.
  3. UI Adjustments:

    • Aligned the size of the locked badge with neighboring icons for a more cohesive appearance.
    • Updated the styles of sidebar buttons to improve user experience and visual consistency.
    • Made adjustments to the selector series styles and icons in the ChromeHeader for better alignment and aesthetics.

These changes collectively enhance the overall styling framework, improve code clarity, and ensure a more consistent user interface across the application.

Copy link

vercel bot commented Jan 26, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gitbutler-components ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 26, 2025 1:47am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
gitbutler-web ⬜️ Skipped (Inspect) Jan 26, 2025 1:47am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant