- ๐ฌ๐งUnited Kingdom catch
Latest change in the MR looks good, but probably needs one more round of manual testing?
- ivnish Poland
btw that after screenshot is that the wrong item?
smustgrave, after screenshot doesn't have wrong warning about cache size
- ๐ช๐ธSpain ckrina Barcelona
First, thanks all for working on this and finding a place that needs improvement. I agree with you all on this being one of those places.
We have restored the warning icon before the deprecated text on deprecated modules, just like in the Seven theme.
Even if this existed in Seven, it doesn't mean it'll visually work in Claro: it results in a really packed, confusing and un-structured group of elements. The solution proposed here is not the right one on a design/UI perspective. The title/heading is designed to hold the title, not title + parenthesis and its text + icons. We can't add info and elements to places that are not designed for that when there are no designs.
The goal of this issue is to make more obvious that a module is obsolete. +1 to that. The next step here is to find the right UI solution for Claro (applies to any UI), and that needs a designer designing that new part of the UI that don't exists (even if that existed in Seven, that solution doesn't work here). When that design is aproved, then let's jump into the implementation. This is the best workflow to be sure we end up with digestible UIs. :)