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 #30969 - replace deprecated mount_react_component #589
Conversation
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.
Thanks @MariaAga ! I believe we need to keep the task-details
class around though.
@@ -15,9 +15,4 @@ | |||
switcher_item_url: foreman_tasks_task_path(:id => ':id') | |||
) %> | |||
|
|||
|
|||
<div class="task-details" id="foremanTaskDetails"> |
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.
We've got some css styles on task-details
namely .task-details pre { ... }
.
I'd choose an easy option and just leave the div, without the id=""
part and put the mount inside.
But we can change css, or add the class into the component itself.
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.
I changed the css so it will be more specific, thanks for the suggestion!
87626d6
to
f40da4c
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.
Thanks @MariaAga 👍 Nice css cleanup ^_^
@adamruzicka could you press the button please? This works nicely and I believe this is ready to get in :) |
With pleasure |
Thank you @MariaAga & @ezr-ondrej ! |
theforeman/foreman#7874
mount_react_component
should be replaced byreact_component