Skip to content

feat: update checkbox and home icon styles #142

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

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

Conversation

nicomiguelino
Copy link
Collaborator

Description

  • Made the check boxes rounded
  • Updated the home icon so that it uses the Screenly logo

Checklist

  • I have performed a self-review of my own code.
  • New and existing unit tests pass locally and on CI with my changes.
  • I have tested my changes on Google Chrome.
  • I have tested my changes on Mozilla Firefox.
  • I added a documentation for the changes I have made (when necessary).

Screenshots

Check box

image

Home icon

image
image
image

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the UI elements for improved style consistency by introducing a custom HomeIcon component and applying rounded styles to checkboxes. Key changes include replacing Bootstrap icons with the HomeIcon component, adding a new round-checkbox class to checkbox inputs, and enhancing warning message styling.

Reviewed Changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/assets/ts/components/sign-in-success.tsx Replaced Bootstrap home icon with the custom HomeIcon component
src/assets/ts/components/settings.tsx Replaced Bootstrap icon with HomeIcon in the settings view
src/assets/ts/components/save-auth-warning.tsx Updated warning text styling for better emphasis
src/assets/ts/components/proposal.tsx Added "round-checkbox" CSS class to style checkboxes as rounded
src/assets/ts/components/home-icon.tsx Introduced a new HomeIcon component based on the Screenly logo
src/assets/ts/components/asset-save-success.tsx Replaced Bootstrap icon with HomeIcon in the asset save success view
Files not reviewed (2)
  • .cursorrules: Language not supported
  • src/assets/scss/style.scss: Language not supported

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