Added Metadata Tooltip Information Modals#707
Conversation
✅ Deploy Preview for dailp ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
| position: "relative", | ||
| display: "inline-flex", | ||
| alignItems: "center", | ||
| marginLeft: "6px", |
There was a problem hiding this comment.
looks great! Only thing I'd suggest is to avoid pixels/px units & using rem instead if possible according to the code standards, but no changes needed here since the tooltips still look nice in smaller screens.
There was a problem hiding this comment.
I would suggest not making a new tsx file for just info tooltips since it will probably be used for metadata only. It might be better to condense files by making InfoTooltip an export of edit-document-modal.tsx and adding the associated styles to edit-document-modal.css.ts
There was a problem hiding this comment.
I would suggest not making a new tsx file for just info tag selectors since it will probably be used for metadata only. It might be better to condense files by making TagSelectoran export of edit-document-modal.tsx and adding the associated styles to edit-document-modal.css.ts
|
Made changes to remove use of pixels in css for tooltips. I did not makes the changes of moving the tooltip login into the metadata as there are potential uses for the component outside of just using it within metadata tooltips. |
2 participants
Added information modal popups as according to task #671 Metadata Information Modals for metadata fields: