Skip to content

remove inline validation error ui+add server-side length guards + card size fixes #724

Merged
moehdotdev merged 1 commit into
mainfrom
home-uiux-changes
May 1, 2026
Merged

remove inline validation error ui+add server-side length guards + card size fixes #724
moehdotdev merged 1 commit into
mainfrom
home-uiux-changes

Conversation

@moehdotdev

@moehdotdev moehdotdev commented May 1, 2026

Copy link
Copy Markdown
Member

remove inline validation error ui + add server-side length guards + card size fixes

what changed

inline rename validation error ui removed
removed all the nameError/titleError state, the live zod validation on every keystroke, and the error <p> elements from every inline rename input: workspace cards, table tabs, workspace header, grid note cards, and list note cards. validation errors were cluttering the ui and the real guard is at save time anyway.

server-side save guards added
added MAX_LENGTH constants and trim + length checks in the settings dropdowns instead:

  • NoteSettings.tsx NOTE_TITLE_MAX_LENGTH = 55, reverts to the original title if invalid
  • TableSettings.tsx TABLE_NAME_MAX_LENGTH = 30, reverts + closes if invalid. also added a useEffect to sync inputValue when tableName prop changes
  • WorkingSpaceSettings.tsx WORKSPACE_NAME_MAX_LENGTH = 30, same pattern

grid note card

  • fixed height h-[179px] so all cards in a row are the same size
  • title font size bumped from text-base to text-lg in both display and edit state
  • grid columns changed from lg:grid-cols-4 to md:grid-cols-4 (shows 4 columns earlier)

inline input styling simplified
with no more error state to conditionally apply, all inline rename inputs now have flat classnames with no cn() toggle.

textarea.tsx
removed md:text-sm from the base textarea lets font size stay consistent at text-base regardless of breakpoint.

note: font-bol typo in the workspace page heading h1 (should be font-bold) snuck in with this pr.

@vercel

vercel Bot commented May 1, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
notevo Ready Ready Preview, Comment May 1, 2026 2:04pm

@moehdotdev moehdotdev merged commit 8283d4f into main May 1, 2026
2 checks passed
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