Skip to content

fix (scrollbar): add border radius to track#182

Open
alex-cerlini wants to merge 2 commits into
levxyca:mainfrom
alex-cerlini:fix-scrollbar-radius
Open

fix (scrollbar): add border radius to track#182
alex-cerlini wants to merge 2 commits into
levxyca:mainfrom
alex-cerlini:fix-scrollbar-radius

Conversation

@alex-cerlini

@alex-cerlini alex-cerlini commented Aug 21, 2024

Copy link
Copy Markdown

Descrição de PR

Adicionado uma borda redonda ao track da scrollbar para melhorar a visualização do card no dark mode.

Issue relacionado

Motivações

Informações adicionais

Após correção:
image

@netlify

netlify Bot commented Aug 21, 2024

Copy link
Copy Markdown

Deploy Preview for diciotech ready!

Name Link
🔨 Latest commit 9002778
🔍 Latest deploy log https://app.netlify.com/sites/diciotech/deploys/66e9e50d1e6e6e0009ca33d9
😎 Deploy Preview https://deploy-preview-182--diciotech.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@levxyca levxyca self-requested a review September 12, 2024 12:26
@levxyca levxyca added the accessibility accessibility-related improvements or additions label Sep 12, 2024
@levxyca levxyca added this to the v2 milestone Sep 12, 2024
@levxyca levxyca linked an issue Sep 12, 2024 that may be closed by this pull request

@levxyca levxyca left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Olá, Alex! Primeiramente, muito obrigada pela sua contribuição 🫱🏻‍🫲🏻

Dando uma revisada, percebi que você modificou o style.css. Como nosso projeto usa Sass, o CSS gerado é compilado a partir do código Sass, então modificar o CSS diretamente pode acabar gerando problemas, já que as mudanças seriam sobrescritas na próxima compilação.

O ideal seria fazer as alterações nos arquivos Sass (com extensão .scss ou .sass) e, depois, recompilar para atualizar o CSS. Aqui está o arquivo Sass principal que estamos usando: https://github.com/levxyca/diciotech/blob/main/assets/sass/style.sass

Você conseguiria ajustar dessa forma? Se precisar de qualquer ajuda, estou à disposição 💙

@levxyca

levxyca commented Apr 24, 2025

Copy link
Copy Markdown
Owner

Oi, @alex-cerlini 💙 Gostaria de entender se você ainda está disponível para trabalhar nesse PR?

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

Labels

accessibility accessibility-related improvements or additions needs update

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Scrollbar dos cards com borda branca

2 participants