-
Notifications
You must be signed in to change notification settings - Fork 284
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
Fixes #36648 - Show desc and publish date on component CV add/edit modal #10685
Conversation
Issues: #36648 |
17dd159
to
54a3b3f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(For posterity): The dropdown seems to be scrolling the modal, but instead should extend outside of it. I think PF has a way you can make this happen.
webpack/scenes/ContentViews/Details/ComponentContentViews/ComponentContentViewAddModal.js
Show resolved
Hide resolved
Updated to make scroll appear on top of modal. The prop was |
ec743f5
to
baf2477
Compare
baf2477
to
a329e32
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a329e32
to
4761317
Compare
4761317
to
0e731df
Compare
0e731df
to
45eb202
Compare
Pushed fixes to tests 🍏 🤞🏼 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking great @sjha4, thanks!
ACK 👍
What are the changes introduced in this pull request?
Add 'published_at' in version label when selecting component CV version to add to composite CV.
Considerations taken when implementing this change?
What are the testing steps for this pull request?
Create some content views and publish. Add descriptions to some/all when publishing. You can also edit desc on published versions.
Create a composite content view and play around with adding/editing/bulk adding components to the composite CV.
Make sure all three(single add/edit and bulk add) look like sketch here: