Skip to content

Made the title clickable to open up dialogue to make changes#217

Merged
jonyTF merged 2 commits intoschej-it:mainfrom
Mickey-Zhaang:UI-UX/click-to-edit-event-title
Mar 31, 2026
Merged

Made the title clickable to open up dialogue to make changes#217
jonyTF merged 2 commits intoschej-it:mainfrom
Mickey-Zhaang:UI-UX/click-to-edit-event-title

Conversation

@Mickey-Zhaang
Copy link
Copy Markdown
Contributor

Simple UI Addition

Title is now hover-able + clickable to open the Dialogue Page to edit the event.
Applicable to other components of the UI -- simple first contribution to start

Recording.2026-02-23.120446.mp4

@maxwellward
Copy link
Copy Markdown
Contributor

Is it possible to make the gray background when hovering only the length of the title text itself? Also, it may be good to use tw-cursor-pointer when hovering over the editable title as well.

It looks like you have some merge conflicts as well which may need to be resolved :)

@Mickey-Zhaang
Copy link
Copy Markdown
Contributor Author

Is it possible to make the gray background when hovering only the length of the title text itself? Also, it may be good to use tw-cursor-pointer when hovering over the editable title as well.

It looks like you have some merge conflicts as well which may need to be resolved :)

Yeah that makes more intuitive sense!

@Mickey-Zhaang Mickey-Zhaang force-pushed the UI-UX/click-to-edit-event-title branch from a19608a to 73f4654 Compare March 16, 2026 15:47
@maxwellward
Copy link
Copy Markdown
Contributor

Great! Tested it locally, looks good to me.

@Mickey-Zhaang
Copy link
Copy Markdown
Contributor Author

So who merges it lol

@jonyTF
Copy link
Copy Markdown
Member

jonyTF commented Mar 31, 2026

that would be me!

@jonyTF jonyTF merged commit 30c223e into schej-it:main Mar 31, 2026
@Mickey-Zhaang Mickey-Zhaang deleted the UI-UX/click-to-edit-event-title branch April 3, 2026 13:56
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