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

feat(settings): add data provider host setting #21

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

jboix
Copy link
Contributor

@jboix jboix commented Mar 12, 2024

Description

This feature enhances the demo application's configurability by allowing users to specify a custom data provider host via the settings page. This flexibility is crucial for adapting the application to different data sources and environments.

Changes made

  • Data Provider Host Setting: Introduced an input field on the settings page for changing the data provider host. This setting is propagated throughout the demo, affecting the player and integration layer callbacks (for the search and list views). The new setting is saved in local storage.
  • Reduced specificity of the theme's input styles to simplify overriding the style.
  • Applied appearance: none to select elements. In Safari browsers this hide default browser-specific appearances, which was showing up until now.

@jboix jboix added the enhancement New feature or request label Mar 12, 2024
@jboix jboix requested a review from amtins March 12, 2024 14:33
@jboix jboix self-assigned this Mar 12, 2024
Copy link

github-actions bot commented Mar 12, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-03-14 16:34 UTC

@jboix jboix force-pushed the feat/data-provider-settings branch 2 times, most recently from de9b2e3 to 4da6701 Compare March 12, 2024 14:40
This feature enhances the demo application's configurability by allowing users to specify a custom
data provider host via the settings page. This flexibility is crucial for adapting the application
to different data sources and environments.

Changes:

- **Data Provider Host Setting:** Introduced an input field on the settings page for changing the
  data provider host. This setting is propagated throughout the demo, affecting the player and
  integration layer callbacks (for the search and list views). The new setting is saved in local
  storage.
- Reduced specificity of the theme's input styles to simplify overriding the style.
- Applied `appearance: none` to select elements. In Safari browsers this hide default
  browser-specific appearances, which was showing up until now.
@jboix jboix force-pushed the feat/data-provider-settings branch from 4da6701 to e5d25ce Compare March 14, 2024 15:58
@jboix jboix requested a review from amtins March 14, 2024 16:00
Copy link
Member

@amtins amtins left a comment

Choose a reason for hiding this comment

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

LGTM! Amazing feature 🚀

@jboix jboix added this pull request to the merge queue Mar 14, 2024
Merged via the queue into main with commit 1abd880 Mar 14, 2024
2 checks passed
@jboix jboix deleted the feat/data-provider-settings branch March 14, 2024 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants