Skip to content

Conversation

ABUDIYAAAA
Copy link
Contributor

@ABUDIYAAAA ABUDIYAAAA commented Sep 23, 2025

Added:

  1. An open recent dropdown under File dropdown (ref [FEATURE] Open Recent sub menu under File menu #585 )
    • Handles no diagrams saved
    • Opening a diagram
    • Displaying tags of lastModified timestamp
    • Updating lastModified timestamp
    • Sorting according to lastModified timestamp
    • Overflow for over 11 children
    • Translation for no saved diagrams in open modal and open recent dropdown

Issues

  • Copied logic of loadDiagram from Workspace
  • Hardcoded overflow height value
  • Translations for the modified open recent and new no diagrams saved
  • Close dropdown when diagram finishes loading

let me know if any changes are required

Screenshot 2025-09-23 at 11 42 45 AM

Copy link

vercel bot commented Sep 23, 2025

@ABUDIYAAAA is attempting to deploy a commit to the dottle's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Sep 23, 2025

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

Project Deployment Preview Comments Updated (UTC)
drawdb Ready Ready Preview Comment Sep 26, 2025 8:55pm

Copy link
Member

@1ilit 1ilit left a comment

Choose a reason for hiding this comment

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

Hey! Thanks for taking the time to work on this

If you have any questions feel free to reach out

@ABUDIYAAAA
Copy link
Contributor Author

ABUDIYAAAA commented Sep 25, 2025

Another concern would be the diagram name being too long and breaking styling. To solve this I can shorten names if that would be the best way to go about it.
Screenshot 2025-09-25 at 2 48 50 PM

Copy link
Member

@1ilit 1ilit left a comment

Choose a reason for hiding this comment

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

Thank you! Left a few minor comments

package.json Outdated
"lexical": "^0.12.5",
"lodash": "^4.17.21",
"luxon": "^3.7.1",
"luxon": "^3.7.2",
Copy link
Member

Choose a reason for hiding this comment

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

why this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Accidentally bumped the version, I have reverted it.

Copy link
Member

@1ilit 1ilit left a comment

Choose a reason for hiding this comment

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

Looks good! Thank you

@1ilit 1ilit merged commit 7359cb7 into drawdb-io:main Sep 26, 2025
3 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.

2 participants