Skip to content

Feature/note detail page#172

Open
SXT230118 wants to merge 22 commits into
developfrom
feature/note-detail-page
Open

Feature/note detail page#172
SXT230118 wants to merge 22 commits into
developfrom
feature/note-detail-page

Conversation

@SXT230118
Copy link
Copy Markdown

@SXT230118 SXT230118 commented Apr 15, 2026

Single notes page design issue; Note page has somewhat matching UI to the Figma. It works but I need to fix some of the proportions.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 15, 2026

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

Project Deployment Actions Updated (UTC)
utd-notebook Ready Ready Preview, Comment Jun 6, 2026 3:47am

Request Review

@SXT230118 SXT230118 linked an issue Apr 15, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Member

@TyHil TyHil left a comment

Choose a reason for hiding this comment

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

Overall this is a great start! Just some things to start changing then hit the rerequest review button when you're done.

Let's use the royal color on light more and cornflower-300 color on dark mode for the accents, icons, and things, to match the rest of the site. And make sure your page works on dark mode too. It's ok to deviate from the design some.

Comment thread src/components/sections/PdfViewer.tsx Outdated
Comment thread src/components/sections/SaveButton.tsx Outdated
Comment thread src/components/sections/SaveButton.tsx
Comment thread tsconfig.json Outdated
Comment thread src/app/notes/[id]/page.tsx Outdated
Comment thread src/app/notes/[id]/page.tsx Outdated
Comment thread src/components/sections/NoteInfoPanel.tsx Outdated
Comment thread src/components/sections/NoteInfoPanel.tsx Outdated
Comment thread src/components/sections/NoteInfoPanel.tsx Outdated
Comment thread src/components/sections/NoteInfoPanel.tsx
…rthe notes. May change when I figure out how the iFrame works.
… still working on that, but I managed to figure out what was requiring the second reload before the pdf showed up.
Copy link
Copy Markdown
Member

@TyHil TyHil left a comment

Choose a reason for hiding this comment

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

Sweet! I think this is almost there! Just some smaller things I noticed to change before it's done

Comment thread src/components/sections/NoteInfoPanel.tsx
Comment thread src/app/notes/[id]/page.tsx Outdated
Comment thread src/components/sections/NoteInfoPanel.tsx Outdated
Comment thread src/components/sections/NoteInfoPanel.tsx Outdated
Comment thread src/components/sections/NoteInfoPanel.tsx Outdated
Comment thread src/components/sections/NoteInfoPanel.tsx Outdated
@TyHil
Copy link
Copy Markdown
Member

TyHil commented Jun 3, 2026

It is summer but wanted to see how work on this is going. I'm going to have a couple weeks before my job starts that I might try and wrap some stuff leftover from the semester up during

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.

Single notes page design

2 participants