Ticket #946 (accepted enhancement)

Opened 4 years ago

Last modified 5 weeks ago

Can not download image when opened in a popup

Reported by: erikos Owned by: humitos
Priority: Low Milestone: 1.0
Component: Browse Version: 0.84.x
Severity: Minor Keywords:
Cc: manuq, humitos, godiard, erikos Distribution/OS: Unspecified
Bug Status: New

Description

When the image is opened in a popup (zoom) I can not get a palette to download it.

Change History

Changed 4 years ago by erikos

Most of the zoomed images open now in a new tab. Only the screenshots in activities.sugarlabs.org do not. In firefox one can only get at the image with: "view background image" and then save the image - which is not straightforward neither.

Changed 4 years ago by erikos

  • status_field changed from Unconfirmed to New
  • type changed from defect to enhancement
  • milestone changed from 0.86 to 0.88

Changed 3 years ago by walter

  • milestone changed from 0.88 to 0.90

As noted, this is painful in all browsers. Maybe 0.90?

Changed 2 years ago by RafaelOrtiz

  • milestone changed from 0.90 to 0.94

Moving to 0.94

Changed 13 months ago by manuq

  • milestone changed from 0.94 to 0.96

Related: #3455 .

Changed 13 months ago by manuq

  • cc manuq added

Changed 13 months ago by erikos

  • milestone changed from 0.96 to 0.98

Not a regression, moving out.

Changed 9 months ago by humitos

  • cc humitos added
  • priority changed from Unspecified by Maintainer to High

Setting priority because it's important.

Changed 8 months ago by humitos

  • priority changed from High to Normal

Well, this should be done by looking if the property background-image exists in the element clicked.

For example, the actual code for  http://activities.sugarlabs.org to show the screenshot images is like:

<div id="lbImage" style="background-image: url(http://activities.sugarlabs.org/en-US/sugar/images/p/5/1235687644); display: block; ">
    <div style="position: relative; width: 700px; height: 525px; ">
        <a id="lbPrevLink" href="#" style="display: none; height: 525px; "></a>
        <a id="lbNextLink" href="#" style="display: none; height: 525px; "></a>
    </div>
</div>

Changing the priority, this is no so important. It's just one specific case.

Changed 8 months ago by humitos

  • cc godiard, erikos added

This is the code we should use to look for background-image property, but we have to go throughout the DOM (from the inside to the top of it) to look that style property.

In the example given in the previous comment, when I click over the image I get the "inner div", so I have to ask for the parent of it to check the background-image property:

>>> element = hit_test.props.inner_node
>>> element.props.style.get_css_text()
'position: relative; width: 700px; height: 525px; '
>>> element.get_parent_element().props.style.get_css_text()
'background-image: url(http://activities.sugarlabs.org/en-US/sugar/images/p/5/1235687644); display: block; '

So, do we want to make that check recursively to check that property? how deep do we want to make it?

Changed 8 months ago by humitos

  • owner changed from erikos to humitos
  • status changed from new to accepted

Changed 8 months ago by dsd

The other consideration to keep in mind is that we will lose DOM access in WebKit2. Might be worth looking at if/how epiphany will cope with this, and/or start some discussion on webkit-gtk list.

Changed 8 months ago by humitos

  • priority changed from Normal to Low

Changed 8 months ago by humitos

  • milestone changed from 0.98 to 1.0

Changed 5 weeks ago by manuq

No other browser is behaving like this. We already make a good job with palettes for images since #4262. I would say won't fix.

Note: See TracTickets for help on using tickets.