- Issue created by @biz123
- πΊπΈUnited States mherchel Gainesville, FL, US
I looked over the code, issue, and looked through the changes
A couple notes:
- This is a large MR
- Lots of order changes, which muddies things up
- Most of the work is in css/base/basic-elements.css and changing the global-text class to wysiwyg
- I don't see anything within the code that's going to break anything.
- Looking visually, I don't see any obvious regresssions
My initial thought is that because 1) This MR has been open for two months and there's a lots of reorganization, this should be merged, because it will hold up other changes.
We should also do a followup to convert the old-style CSS to use nesting.
Not yet setting this to RTBC, because not sure if @biz123 thinks its ready (it's not set to review).
- πͺπΈSpain fjgarlin
In the feedback round, it was mentioned that font size was still too big. Are we addressing this here?
- πΊπΈUnited States mherchel Gainesville, FL, US
Good catch!
@biz123 - Let me know if you're still actively working on this. If not, I'm happy to help