We have around 35 occurrences where we use setStyle(..) in JabRef. The problem is that this usually hardcodes colors or style like insets. Therefore, custom themes (CSS) or other stylesheets like Atlanta will always be overwritten by this inline styles.
Priority 1:
- Remove all
setStyle(..) calls that set a color. Those are the most critical one we should get rid.
Priority 2:
- Remove all remaining
setStyle(..) calls. Use generic style classes, like a .bold style class for bold text. We can reuse that and are good to go
Additional context
Example:
We have around 35 occurrences where we use
setStyle(..)in JabRef. The problem is that this usually hardcodes colors or style like insets. Therefore, custom themes (CSS) or other stylesheets like Atlanta will always be overwritten by this inline styles.Priority 1:
setStyle(..)calls that set a color. Those are the most critical one we should get rid.Priority 2:
setStyle(..)calls. Use generic style classes, like a.boldstyle class for bold text. We can reuse that and are good to goAdditional context
Example:
