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

Feature/home screen #63

Open
wants to merge 14 commits into
base: version/0.2.0
Choose a base branch
from
Open

Feature/home screen #63

wants to merge 14 commits into from

Conversation

tonelli-m
Copy link
Collaborator

Description

Implementing home screen following designs from Figma.

Currently only the header part with sound level meter is implemented, other sections will be added in later pull requests.

Changes

  • Move sound level meter header from MeasurementScreen into a separate component so it can be reused
  • AudioSource is now managed at coordinator level so it can be uninterrupted between the home and measurements screens
  • Added a new NCButton class to manage button styling in a unified manner
  • Settings are now accessible through a cogwheel icon in the upper right corner
  • Updated noise level color ramp to use the palette from NoiseModeling (contrasts are not always great when using these colors on a white background so we might think about slightly adapting those colors)
  • Did a bunch of cleanup and refactoring here and there

Linked issues

#62

Checklist

  • Code compiles correctly on all platforms
  • All pre-existing tests are passing
  • If needed, new tests have been added
  • Extended the README / documentation if necessary
  • Added code has been documented

@tonelli-m tonelli-m added the KMP Shared Kotlin code label Jan 16, 2025
@tonelli-m tonelli-m added Android Android related code iOS iOS related code labels Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android Android related code iOS iOS related code KMP Shared Kotlin code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants