EncyclopediaOfLife/eol — comment on commit d2672806882342e36b5681d0d7dd7da7dc58cdfd
File: app/views/admins/statistics/user_added_text.html.haml line 2
By lwalley:

Note the #content_partners id here. This is not a content partner view so we shouldn't use the id from that view. CSS IDs should be named so that they are meaningful in their context, this ID name is not meaningful to the context of the statistics and is confusing here. HR's approach was to have unique IDs and sometimes CSS for every view, whilst it is sometimes useful to have a unique ID for when exceptions arise, if there are no exceptions to the common shared CSS then a unique ID is unnecessary. I think we may want to think about the commonality between views and sharing a bit more of the CSS. We can pair on this and other styling issues raised in this commit.