Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance/ux #6

Closed
wants to merge 5 commits into from
Closed

Enhance/ux #6

wants to merge 5 commits into from

Conversation

HDDTHR
Copy link

@HDDTHR HDDTHR commented Oct 5, 2023

Here are the changes I made:

  • Replaced the custom note list components with the Accordion component from the @strapi/design-system package.
  • Changed the import mechanisms to be able to use develop --watch-admin without errors
  • Added logic to return and display the date of creation and the date of update along with the user for each note

@ComfortablyCoding
Copy link
Contributor

Thanks! I'll try and have a look at it over the weekend.

@ComfortablyCoding
Copy link
Contributor

Unfortunately while I do like the concept of this enhancement it results in the view being squished which degrades the user experience (see sample below). Due to this I will be rejecting the PR.

Current
Screenshot from 2023-10-10 22-54-24

Proposed
Screenshot from 2023-10-10 22-45-57

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.

None yet

2 participants