Skip to content
This repository has been archived by the owner on Dec 13, 2020. It is now read-only.

Tooltip on first level breadcrumb #683

Closed
metas-mk opened this issue Apr 25, 2017 · 12 comments
Closed

Tooltip on first level breadcrumb #683

metas-mk opened this issue Apr 25, 2017 · 12 comments

Comments

@metas-mk
Copy link
Member

Type of issue

Feature request

Current behavior

Currently, the breadcrumb navigation does not have a tooltip when hovering.

Expected behavior

Show a tooltip when hovering w/ the same delay as other tooltips. The tooltip shall describe what the user shall expect when pressing.

image

@metas-mk metas-mk added this to the 2017-18 milestone Apr 25, 2017
@Dunkat Dunkat self-assigned this Apr 25, 2017
Dunkat added a commit that referenced this issue Apr 26, 2017
Dunkat added a commit that referenced this issue Apr 26, 2017
damianprzygodzki added a commit that referenced this issue Apr 26, 2017
Tooltip on first level breadcrumb #683
metas-ts added a commit to metasfresh/metasfresh that referenced this issue Apr 28, 2017
[#332](metasfresh/metasfresh-webui-api-legacy#332) Empties window without pre-selection possible
[#695](metasfresh/metasfresh-webui-frontend-legacy#695) Lager photo preview screen/ modal overlay
[#328](metasfresh/metasfresh-webui-api-legacy#328) Manufacturing receipt mandatory Field not filled
[#1388](#1388) HU from Cost Collector is locked and cannot be selected anymore after the CC was reversed
[#683](metasfresh/metasfresh-webui-frontend-legacy#683) Tooltip on first level breadcrumb
[#682](metasfresh/metasfresh-webui-frontend-legacy#682) Sidelist opening shall automatically have the first line selected
[#322](metasfresh/metasfresh-webui-api-legacy#322) Manufacturing Issue/Receipt: Fix Qty and QtyPlan columns
[#676](metasfresh/metasfresh-webui-frontend-legacy#676) Process result: implement openDocument as modal
[#1358](#1358) Drop de.metas.order.callout.CheckDouble
[#411](metasfresh/metasfresh-webui-frontend-legacy#411) Make the parameter names visible in filters
@metas-lc
Copy link

metas-lc commented May 3, 2017

IT
there is no tooltip on the first level breadcrumb NOK

Dunkat added a commit that referenced this issue May 4, 2017
Fix Tooltip on first level breadcrumb #683
@teosarca
Copy link
Member

teosarca commented May 4, 2017

Quick checked, can u pls hide "the other" tooltips because are pointless?
image

image

Dunkat added a commit that referenced this issue May 5, 2017
Dunkat added a commit that referenced this issue May 5, 2017
@cadavre
Copy link
Contributor

cadavre commented May 10, 2017

#683 (comment) There's no need for tooltips in any other breadcrumb element than pointed out by Mark in issue description. The only breadcrumb element that changes STATE of window is last clickable – which goes to "documents list", other simply opens overlays which you can simply deactivate.

Please – don't overflow UI with unnecessary elements.

PS @teosarca system default tooltips were there because some long names might not fit in breadcrumb – tooltips were used to make them readable. With new assumption of "document name" removal from breadcrumb I think it might be better, but still – you requested it half year ago for a legit purpose. :)

@metas-lc metas-lc self-assigned this May 10, 2017
@metas-lc
Copy link

IT

  1. sales/ purchase order, hover on the first level breadcrumb
    => tooltip: "Go to default documents list" OK
    => default tooltips are not showing OK
    => (regression) fields still have default tooltip OK
    => works the same in other windows OK

@metas-lc
Copy link

i noticed that the other tooltips take about 1s to appear, while this one appears instantly. NOK
Can you please put a delay on this one as well?

@Dunkat
Copy link
Contributor

Dunkat commented May 11, 2017

this one tooltip is an exception and should appears instantly, because user should notice it before click in link

@cadavre
Copy link
Contributor

cadavre commented May 11, 2017

Instant appear should ONLY have n-1 breadcrumb – the one that leads to other page with documents list.

@metas-mk
Copy link
Member Author

@cadavre @Dunkat I don't think it's an exception, why?
the tooltip for the n-1 breadcrumb just describes what the user will expect when clicking on it, same as in Action menu, Navigation, Inbox and so on. the user will want to read this once/ twice and then knows it forever. why should the tooltip then pop up everytime she/ he clicks on the breadcrumb?
I have a strong feeling that the delay should be harmonized on this n-1 breadcrumb too.

@cadavre
Copy link
Contributor

cadavre commented May 11, 2017

I guess the idea behind that was if users clicks it – it will redirect page and loose filters/sorting/ordering/page... As for me keeping it unified (all with delay) is ok @metas-mk . :)

@metas-mk
Copy link
Member Author

right, perfect, thanks.
then let's go with delay there too.
@Dunkat can u please take over from here ;-)

@teosarca
Copy link
Member

moving it back to Development.

@metas-lc
Copy link

IT
checked again. works fine
there is a delay on every tooltip

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants