- Issue created by @pameeela
I have installed the module but how to view these button could you let me know.
- Merge request !21Issue #3494403: Add some margin or padding to the resume/discard modal → (Merged) created by Unnamed author
- 🇦🇺Australia pameeela
Thanks @sandip poddar, I think we probably need a different solution as the id will vary, e.g. for my site it is
#ui-id-3
. I haven't looked but perhaps we can add a specific class to the modal or the div and use that to apply styles. - 🇮🇳India sandip
Yes @pameeela, I am looking into it to add specific class to the model.
- 🇮🇳India sandip
Please have a look into it. I have added specfic class to the modal and it is working fine.
- 🇦🇺Australia pameeela
Thanks, made some small tweaks because the extra side padding put the text out of alignment with the buttons. I didn't find that the
!important
was needed so removed that too. And added a max-width for readability on larger screens. - 🇮🇳India sandip
Hi @pameeela,
I believe this issue has been resolved. If you agree, we can move it to RTBC. Thank you! - 🇦🇺Australia pameeela
Since we both worked on it, generally we would not set it to RTBC. That should be done by a reviewer who has not worked on it. But I do think this is ready, in my opinion it is fixed.
- 🇮🇳India sandip
Thanks for clarifying the RTBC process. I appreciate your guidance!
- First commit to issue fork.
-
hchonov →
committed a7dbd901 on 8.x-1.x authored by
sandip poddar →
Issue #3494403: Add some margin or padding to the resume/discard modal
-
hchonov →
committed a7dbd901 on 8.x-1.x authored by
sandip poddar →
- 🇩🇪Germany hchonov 🇪🇺🇩🇪🇧🇬
Thank you, I just merged it as it looks quite better now!
Before:
After:
Automatically closed - issue fixed for 2 weeks with no activity.