fecgov / openFEC-web-app

DEPRECATED See https://github.com/18F/fec-cms for fec.gov's code
Other
43 stars 31 forks source link

Implement new designs for "Legal resources" landing page #2288

Closed jenniferthibault closed 6 years ago

jenniferthibault commented 7 years ago

Discussion and history in https://github.com/18F/openFEC-web-app/issues/2185


Designs

screen shot 2017-10-25 at 10 26 59 am

Notes

Links

Tasks to completion:

AmyKort commented 7 years ago

@dorothyyeager and @gsmaragdis -- just flagging this one for you.

jenniferthibault commented 6 years ago

Image above edited to include final icon designs.

Some of these icons have already been added:

The rest of the icons are exported as SVG's and packaged in this zip file: 📁 legal-icons.zip

johnnyporkchops commented 6 years ago

@jonella, @jenniferthibault - Do you think the icon SVGs should al start at the same size for cards? We can, of course, set differing background size properties in the CSS to accommodate the different originals sizes. For example the ones on data landng inherit the card__icon background size of 50%. If I add the new ones, I have to add a specific rule for 75% for the ones that appear on legal resources landing. If these were the same size it would mean fewer lines of CSS , not big deal but thought it might be a smart coordination between design/front-end teams. smaller_svg

johnnyporkchops commented 6 years ago

Disregard the last comment regarding icons, I just checked and realized that the new icons made were the same size as most. Just a few older ones are smaller, which is not worth changing, given that they may have already been used throughout the SCSS with various bg-size properties, ok_svg

jenniferthibault commented 6 years ago

@johnnyporkchops our working standard is to build each icon at a 20x20 px square so that the line weights and spacing are consistent when scaled. You're right that a few of the earlier ones are probably not consistent to that template.

Since the gavel is being replaced overall, here it is exported as 20x20, so we can make it match. gavel.svg.zip

Since not all icons have perfectly square proportions, when exporting we crop the artboard to the dimensions of the art. Perhaps what you're saying is that we shouldn't do this, and instead should export all icons at 20x20, including white space outside the artwork? @xtine what do you think?

Here are some examples from the working file, where you can see that the checklist, CFR and U.S.C. icons aren't full 20x20: screen shot 2017-10-30 at 10 34 39 am

johnnyporkchops commented 6 years ago

The non-square seem to work fine in the card icons even though they have a different height, I was more concerned with the older, smaller sizes. I will replace mallet with gavel at the new size. Thanks!

xtine commented 6 years ago

@jenniferthibault: personally I'm not concerned with all icons being 20x20 because each icon needs a specific class defined for them, so the positioning is defined in the stylesheet.

patphongs commented 6 years ago

Had a question for @jenniferthibault regarding the card design on the legal resources landing page: https://github.com/18F/fec-cms/pull/1443.

AmyKort commented 6 years ago

Issue moved to 18F/fec-cms #1601 via ZenHub