Landmark icon

[connium]

Icon Request

• Icon name: bank, landmark
• Use case: bank, building, business, capitol, landmark
• Screenshots of similar icons:

In #347 and #350 bank icons are provided. AFAICS the icons do not fully match the Icon Design Guide.

I adapted the proposed icons, but I'm not quite sure which one fits best. So I present them here before creating a PR. Any hints are welcome!

1. Square base, 1px spacing
   

2. Square base, 2px spacing
   

3. Single base line, 2px spacing
   

4. Double base line, 1px spacing
   

5. Double base line, 2px spacing
   

6. Double base line, 2px spacing, 1px baseline spacing (one could argue that the two base lines belong to the same structural element)
   

Please let me know for which one I should create a PR.

[BeezBeez]

Thanks for the time spent on the bank icon !

[BeezBeez]

I personally prefer the 4th icon (Double base line, 1px spacing)

[ghost]

the icon guidelines actually say that

Distinct elements must have 2 pixels of spacing between each other.

so I would go with the 5

[ericfennis]

I like number 4th or the 5th one the most.

[lukas-schaetzle]

I like number 6 the most.

[ghost]

but the 1, the 4 and the 6 break the 2px spacing rule

[connium]

The guidelines say that

Distinct elements must have 2 pixels of spacing between each other.

I created the 1px versions because to me it is not clear what is meant with "distinct elements".

• If each SVG element, that does not touch another element, counts as a distinct element, then 2, 3, 5 are the only valid ones.
• If base, pillars and roof of the building count as distinct elements, then 2, 3, 5, 6 are valid.
• If the building itself is a distinct element (in this case a building and a mountain in the same icon are two distinct elements), then all variants adhere to the guidelines.

Since I am unsure what is meant to be a distinct element and which is the best variant, I leave the decision to the community ;-)

[ghost]

I think that each path that does not touch another element counts as a distinct element

[ericfennis]

What @delnyn says. So 2,3 and 5 are valid according to the guidelines. Maybe 6 can be allowed (because the stairs)🤔 , but that's only if it is a 'better' option.

[ericfennis]

Are the icons from #347 and #350 also in this list?

[ghost]

my favorite is the 3

[connium]

Are the icons from #347 and #350 also in this list?

Not exactly. The one from #347 is similar to 4 but doesn't use the full height of the view box. The icon from #350 is close to 2.

[connium]

Counting the votes in the comments before gives me

• 3: 1 vote
• 4: 2 votes
• 5: 1 vote
• 6: 1 vote

Considering that icon 4 is not a valid option, I still don't know for which icon I should create a PR 😕

[ghost]

I really think the 3 fits better with the rest of the set

[ericfennis]

@connium We discussed this in the discord channel, and we're in for number 3. You okay with that?

[connium]

@ericfennis I'm totally fine with this decision. Thanks for taking care of it!

I just created a pull request for the icon.