-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
543 fixing Plot setting panel icon #181
Conversation
Code Coverage Summary
Diff against main
Results for commit: 69d0132 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Something isn't right. The first click does not affect the icon. plot_settings.mp4 |
I am not able to reproduce this. I am testing in chrome in mac and window Screen.Recording.2023-08-21.at.6.26.02.PM.movScreen.Recording.2023-08-21.at.6.30.16.PM.mov |
I am on Ubuntu 20.04 and use Brave, which is Version 1.57.47 Chromium: 116.0.5845.96 (Official Build) (64-bit) |
Understood, it appears that the problem is limited to Bootstrap version 5. It seems to be functioning correctly in versions 3 and 4. Can you please confirm if you are using Bootstrap version 5? I am only able to reproduce the issue in BS 5. |
Actually, it works fine with when I set |
That's quite intriguing! My system is actually experiencing the complete opposite. 😄. I might have identify the issue but I'm uncertain how it's impacting various versions on different systems. let me fix all version in my system and hopefully it will fix in all system aswell. |
When using To fix this I have used event delegation here. There is a similar implementation called DEMO Screen.Recording.2023-08-23.at.7.06.46.PM.mov |
Merge branch 'main' into 543_panel_icon@main # Conflicts: # tests/testthat/_snaps/app_pws_ui/plot_with_settings_ui/pws-download_menu.png # tests/testthat/_snaps/app_pws_ui/plot_with_settings_ui/pws-hidden.png # tests/testthat/_snaps/app_pws_ui/plot_with_settings_ui/pws-resize_menu.png # tests/testthat/_snaps/app_pws_ui/plot_with_settings_ui/pws-visible.png
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.
Yup, that's better 👍
this fixes insightsengineering/teal.modules.general#543
Here I have added Jquery to toggle icon and added class card-heading to identify target div() bs4/bs5 version.