Bug 459787 - [nav][hover] Investigate removing the images from the navigation hovers
Summary: [nav][hover] Investigate removing the images from the navigation hovers
Status: CLOSED WONTFIX
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: JS Tools (show other bugs)
Version: 8.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Michael Rennie CLA
QA Contact:
URL:
Whiteboard:
Keywords: performance
Depends on:
Blocks: 478357
  Show dependency tree
 
Reported: 2015-02-12 09:58 EST by Michael Rennie CLA
Modified: 2017-01-10 15:42 EST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Rennie CLA 2015-02-12 09:58:39 EST
This will apply to CSS and HTML hovers as well - but we should look at alternate presentations for the hover links since the file type image in the hover causes another GET to be done for each link, each time the hover is shown. This has a negative impact on the speed that the hover shows on slower networks.

We should see if:

1. there is another way to use an image (without using an <img> tag)
2. we can make it look just as snazzy without an image
Comment 1 Eric Moffatt CLA 2015-02-12 15:56:59 EST
Mike, could this be one of the reasons the defers take longer than expected ?
Comment 2 Michael Rennie CLA 2015-02-12 16:04:04 EST
(In reply to Eric Moffatt from comment #1)
> Mike, could this be one of the reasons the defers take longer than expected ?

I would have to say yes. And when you couple that with a latent network, you get all kinds of fun side effects.
Comment 3 Michael Rennie CLA 2015-05-14 10:50:57 EDT
We should have a look at what navigatorRenderer#addimageToLink does, then we could maybe create our links using the imageClass for the content type we are wanting to nav to. 

I don't know if this is even doable since we produce markdown to send back, and I don't know if theres even a way to specify CSS stuff in markdown.
Comment 4 Michael Rennie CLA 2017-01-10 15:42:16 EST
Closing as part of a mass clean up of inactive bugs. Please reopen if this problem still occurs or is relevant to you. For more details see:

https://dev.eclipse.org/mhonarc/lists/orion-dev/msg04002.html