Skip to content

Resolve "Icons on forks page are to big"

Simon Knox requested to merge 42046-fork-icon into master

What does this MR do?

Set size for fork icon fix fork button icon color

Are there points in the code the reviewer needs to double check?

should we set a default size for the sprite_icon method? icon.vue currently sets default of 16 - this means forgetting to set a size could mean a slightly-too-large icon rather than a very-obviously-broken one

Screenshots (if relevant)

Before:

Screen_Shot_2018-01-16_at_6.40.01_am_copy

After:

Screen_Shot_2018-01-16_at_6.32.10_am_copy

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #42046 (closed)

Merge request reports