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

refactor: enhance configuration url handle #499

Merged
merged 1 commit into from
Nov 17, 2024

Conversation

Revyn112
Copy link
Contributor

Fixes #[issue_no]

Summary of Changes

This update enhances the "Developer Settings" UI by introducing a new Reload Installer button for reloading the application and replacing the existing Reset to Default button with a styled Button component, ensuring a consistent and modern user experience. The changes also improve layout alignment and maintain existing functionality.

Changelog

Added

  • Reload Installer Button:
    • Introduced a new button using the Button component.
    • Styled with ButtonType.Neutral and class attributes (ml-2 h-fit min-h-10 text-lg) for consistent UI.
    • On click, sends the ipcRenderer.send(channels.window.reload) command to reload the installer.

Modified

  • Reset to Default Button:
    • Replaced the existing <input type="button"> element with the Button component.
    • Preserves functionality to reset the "Configuration Download URL" to packageInfo.configUrls.production.
    • Styled using ButtonType.Neutral and class attributes (ml-2 h-fit min-h-10 text-lg) for uniformity.

Screenshots (if necessary)

Before:

381765242-910feee1-f091-450d-9800-e74be3c4f7dd

After:
grafik

Additional context

Discord username (if different from GitHub):

@frankkopp frankkopp merged commit e652665 into flybywiresim:master Nov 17, 2024
3 checks passed
frankkopp pushed a commit that referenced this pull request Nov 17, 2024
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.

2 participants