- Issue created by @alok_singh
- Merge request !4flexistyle_dxp-3493749: On mobile view tag has wrong styling needs to be fixed. → (Open) created by alok_singh
Verified MR4, the tags are styled and the space issue is resolved. RTBC++
- 🇮🇳India anoopsingh92 Rajasthan, India
Hi @alok_singh, thanks for creating the MR!
Just a quick note regarding the use of
!important
in the CSS. It's generally best practice to avoid using!important
unless absolutely necessary, as it can make the styles harder to maintain and debug.For this case, we could consider a more specific selector or adjusting the CSS cascade to achieve the desired result without
!important
. Let me know if you'd like any help with that!Thanks!
- 🇮🇳India anoopsingh92 Rajasthan, India
Hi @alok_singh,
Since the theme is based on Bootstrap, it would be ideal to leverage Bootstrap’s built-in classes instead of writing custom CSS. For instance, you can use the responsive margin and display utilities available in Bootstrap, such as:
.flex-article .tags { margin-bottom: 20px; } @media (max-width: 767px) { .flex-article .tags { display: block !important; } }
Instead, try using Bootstrap classes like mb-3 for margin and d-block for the display property on smaller screens.
This will ensure better alignment with Bootstrap’s best practices and keep the code cleaner.