Skip to content

docs: add native frontend architecture implementation plan#70

Open
outlaw-dame wants to merge 5 commits into
masterfrom
frontend-native-ui-docs
Open

docs: add native frontend architecture implementation plan#70
outlaw-dame wants to merge 5 commits into
masterfrom
frontend-native-ui-docs

Conversation

@outlaw-dame

Copy link
Copy Markdown
Collaborator

Summary

Adds an internal implementation document for migrating Memory's frontend toward a Framework7-first, OS-adaptive, native-feeling UI architecture.

What changed

  • Adds .gitignore coverage for docs/internal/ so future local internal planning notes are not accidentally added.
  • Adds docs/internal/frontend-native-architecture-implementation.md as the reviewed implementation reference.
  • Documents the proposed architecture around Vue 3, Framework7, Capacitor, Tailwind as token/layout utility, and Iconoir behind a semantic icon adapter.
  • Includes OS/capability detection guidance for leaning on native icons where legally/practically available, native emoji, native keyboard behavior, haptics, safe areas, share sheets, PWA display modes, gestures, and motion.
  • Includes phased migration plan, testing strategy, accessibility/security requirements, dependency policy, and acceptance criteria.

Notes

This PR is documentation-only. The branch originally had one interrupted placeholder file; it was removed before opening this PR, and the final diff only contains .gitignore plus the intended implementation document.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a comprehensive implementation plan for migrating the frontend to a Framework7-first, OS-adaptive semantic architecture, and updates the .gitignore file to exclude internal documentation drafts. Feedback on the plan identifies an invalid HTML attribute value for enterkeyhint ("newline") in the composer examples, which should be corrected to "enter" to align with the HTML specification.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Post composer:

```html
<textarea enterkeyhint="newline" autocapitalize="sentences" autocorrect="on" spellcheck="true"></textarea>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

According to the HTML specification, newline is not a valid value for the enterkeyhint attribute. The allowed values are enter, done, go, next, previous, search, and send. To indicate inserting a new line, use enter instead.

Suggested change
<textarea enterkeyhint="newline" autocapitalize="sentences" autocorrect="on" spellcheck="true"></textarea>
<textarea enterkeyhint="enter" autocapitalize="sentences" autocorrect="on" spellcheck="true"></textarea>

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