search

saurabhd / hk_realestate

GNU General Public License v2.0
1 stars 0 forks source link

Add phone icon with number to service menu #29

Closed heliogabal closed 8 years ago

heliogabal commented 8 years ago

We need an addition to the service menu: A telephone icon that reveals the main telephone number when hovered: servicemenu_phone

icon_telephone

darshi-shah commented 8 years ago

Hi Rainer,

These changes are pushed on develop branch. Please take pull and revert hk_crm feature.

Thanks Darshi

heliogabal commented 8 years ago

Hi Darshi, maybe I didn't explain myself precisely enough... the phone icon padding was meant to be in the desktop view as well. If you compare the first screenshot with your work, you will realize that the icons are cramped together and don't have enough space between them to be nicely selectable:

auswahl_027

so the icons in desktop view should have a padding to their left (maybe 0.5em?) to make the clickable surface bigger and give them some withespace.

Then, on Tablet view, the icons should be evenly distributed and far enough away from each other to be easily selected with a finger. This is what it looks like now to me:

auswahl_028

So there is not enough space between favorite and phone, and the language selector is too far away.

When I select the phone icon on iPad, the tooltip doesn't show correctly:

auswahl_029

On iPhone, its better and the tooltip shows, but the language icon is still too far down:

auswahl_030

Hope that makes it more clear...