- Issue created by @shweta__sharma
- Assigned to bobi-mel
- Merge request !11fepper-3390535:Added padding to the new text in comment. → (Closed) created by shweta__sharma
- Status changed to Needs review
about 1 year ago 6:34am 29 September 2023 - Issue was unassigned.
- 🇺🇦Ukraine bobi-mel
In the theme, the main CSS Units are the 'em' I suggest using them as the general units in all cases.
- 🇺🇦Ukraine bobi-mel
In the theme, the main CSS Units are the 'em' I suggest using them as the general units in all cases.
correct patch
- Status changed to RTBC
about 1 year ago 9:36am 2 October 2023 - 🇵🇭Philippines Zed9
Hi bobi-mel,
I've tested the patch and can confirm that the padding was added. Please see attached image.
- 🇺🇸United States e2tha-e
This issue isn't a bug or breakage. While suggestions for aesthetic improvement are greatly appreciated, Fepper Theme is meant to be a base theme, where such improvements are meant to be implemented in a sub-theme.
This is how things look without the patch:
It's a case of being good enough for everyone, where perfection should be up to each implementer.
- Status changed to Closed: works as designed
7 months ago 1:44pm 13 April 2024