Skip to content

Refactor components for improved font usage and maintainability - #1

Merged
BennyGK15 merged 2 commits into
mainfrom
dev
Mar 16, 2026
Merged

Refactor components for improved font usage and maintainability#1
BennyGK15 merged 2 commits into
mainfrom
dev

Conversation

@BennyGK15

Copy link
Copy Markdown
Owner

This pull request introduces a major refactor of the ExportDialog component, splitting its logic into smaller, reusable modules and utility functions. It also updates font variable usage for improved consistency across the app, and makes minor UI improvements to the SectionEditor component.

Export Dialog refactor and modularization:

  • The original components/export-dialog.tsx file was removed and replaced with a new modular structure: ExportDialog, ExportDialogActions, and utils were separated into their own files under components/export-dialog/. The dialog logic, actions, and utility functions are now clearly separated for better maintainability and readability. [1] [2] [3] [4] [5]
  • Utility functions for filename generation, content statistics, clipboard copying, and file downloading were moved to components/export-dialog/utils.ts, making them reusable and easier to test.

Font variable and layout improvements:

  • The font variable for the Inter font was renamed from --font-sans to --font-inter in both app/layout.tsx and app/globals.css, and applied directly to the <html> element for improved global font handling. [1] [2] [3]

UI and accessibility enhancements:

  • The SectionEditor component now uses wider dropdowns for section type and number selection, improving usability. [1] [2]
  • Minor formatting improvements were made to the metadata and content in app/ki-hinweis/page.tsx for better readability. [1] [2]

@BennyGK15
BennyGK15 merged commit 02df426 into main Mar 16, 2026
1 check passed
@sonarqubecloud

Copy link
Copy Markdown

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