Skip to content

fix: style IMP Widget UI similar to sec reader#36

Merged
ElectricCookie merged 6 commits intodevfrom
fix/ui-similar-to-sec-reader
Oct 13, 2025
Merged

fix: style IMP Widget UI similar to sec reader#36
ElectricCookie merged 6 commits intodevfrom
fix/ui-similar-to-sec-reader

Conversation

@flxapps
Copy link

@flxapps flxapps commented Oct 8, 2025

@flxapps flxapps self-assigned this Oct 8, 2025
Copilot AI review requested due to automatic review settings October 8, 2025 09:04
Copy link

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 IMP Widget UI to align with the styling used in the security reader. The changes focus on making the UI more consistent across the codebase by updating button styles, spacing, and layout components.

  • Replace standard buttons with LdButtonVague components with consistent styling
  • Update text components to use muted styling where appropriate
  • Modernize Android Gradle configuration files across example projects

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.

File Description
lib/src/ui/imp_widget.dart Updated UI components to use consistent button styling, muted text, and improved layout
lib/src/imp_reader.dart Minor code formatting improvements for line length consistency
example//android/ Modernized Android Gradle configuration files and updated dependencies

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@flxapps flxapps force-pushed the fix/ui-similar-to-sec-reader branch from 0a2feb7 to 9246e13 Compare October 8, 2025 09:24
Copy link

@ElectricCookie ElectricCookie left a comment

Choose a reason for hiding this comment

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

Very nice, please only adjust the wording around the remaining readings left per token, to the message "Internet connection ... " as the end user does not know what tokens are.

@ElectricCookie ElectricCookie merged commit 6ebd29b into dev Oct 13, 2025
9 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.

3 participants