Skip to content

Conversation

Aniket-Engg
Copy link
Collaborator

fixes #6234
fixes #6230 (comment)

Copy link

netlify bot commented Aug 12, 2025

Deploy Preview for remixproject ready!

Name Link
🔨 Latest commit 27b3ad7
🔍 Latest deploy log https://app.netlify.com/projects/remixproject/deploys/689dc443eab762000871e7a5
😎 Deploy Preview https://deploy-preview-6249--remixproject.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 project configuration.

@Aniket-Engg Aniket-Engg requested a review from joeizang August 12, 2025 12:18
@Aniket-Engg Aniket-Engg force-pushed the bs5darkfixes2 branch 2 times, most recently from 3aa18ad to 9bbc4ba Compare August 12, 2025 13:18
@joeizang
Copy link
Collaborator

joeizang commented Aug 12, 2025

  • Margins on the outside are still very small. From the image, you see that the input field is sitting right on the terminal.
image
  • This caret should have the right color, see the image below:
image
  • Border is the wrong color.
image
  • The borders and button text colors need to be fixed.
image
  • Margins between elements is off.
image
  • Some buttons don't have the right text color:
image
  • Width adjustment seems to be broken now. This is not the case on remix.ethereum.org
image

This is on the preview link:
image

  • Error cards seem to be the wrong size
Screenshot 2025-08-12 at 15 51 50

@Aniket-Engg Aniket-Engg added the ready-to-review PR ready to review label Aug 13, 2025
@Aniket-Engg Aniket-Engg merged commit 6a87a89 into master Aug 14, 2025
29 of 31 checks passed
@Aniket-Engg Aniket-Engg deleted the bs5darkfixes2 branch August 14, 2025 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-review PR ready to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fixes to Bootstrap 5 Dark Theme Bootstrap 5 improvements
2 participants