Skip to content

docs: add frontend guidelines#567

Open
timothejoubert wants to merge 7 commits into
mainfrom
feature/guidelines
Open

docs: add frontend guidelines#567
timothejoubert wants to merge 7 commits into
mainfrom
feature/guidelines

Conversation

@timothejoubert

@timothejoubert timothejoubert commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Especially dedicated to external dev

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new GUIDELINES.md document to standardize frontend contribution practices for this Nuxt/Vue codebase.

Changes:

  • Introduces a comprehensive frontend guidelines document covering DOM structure, component naming, CSS Modules/SCSS rules, accessibility, Vue/TS conventions, i18n, performance, and a PR checklist.
  • Documents CSS Modules class naming/modifier patterns and guidance around CSS custom properties for theming/overrides.
  • Adds accessibility and performance recommendations tailored to the project’s component patterns.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread GUIDELINES.md Outdated
Comment thread GUIDELINES.md Outdated
Comment thread GUIDELINES.md Outdated
@timothejoubert timothejoubert marked this pull request as draft June 4, 2026 14:31
- Soften "no global utility classes" to avoid contradiction with `.visually-hidden`
- Replace broken "Themes section" reference with actual mixin file path
- Fix lodash import example (default import, not named import)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@timothejoubert timothejoubert marked this pull request as ready for review June 4, 2026 14:35
Comment thread GUIDELINES.md Outdated
Comment thread docs/GUIDELINES.md
Comment thread GUIDELINES.md Outdated
Comment thread GUIDELINES.md Outdated
Comment thread docs/GUIDELINES.md
Comment thread GUIDELINES.md Outdated
Comment thread GUIDELINES.md Outdated
Comment thread GUIDELINES.md Outdated
Comment thread GUIDELINES.md Outdated
Comment thread GUIDELINES.md Outdated
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