Page MenuHomePhabricator

Make clicking on file icon in lightbox download file
ClosedPublic

Authored by chad on Dec 2 2016, 6:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 2, 7:18 PM
Unknown Object (File)
Wed, Feb 26, 1:04 AM
Unknown Object (File)
Sat, Feb 22, 4:47 PM
Unknown Object (File)
Tue, Feb 18, 12:08 PM
Unknown Object (File)
Feb 12 2025, 12:41 AM
Unknown Object (File)
Feb 9 2025, 2:55 AM
Unknown Object (File)
Jan 29 2025, 10:34 PM
Unknown Object (File)
Jan 29 2025, 11:23 AM
Subscribers

Details

Summary

Ref T3612, this adds a anchor around the large icon with hover state so you can download from here as well.

Test Plan

Hover over .ics file, click, get download.

Diff Detail

Repository
rP Phabricator
Branch
download-ics (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14737
Build 19259: Run Core Tests
Build 19258: arc lint + arc unit

Event Timeline

chad retitled this revision from to Make clicking on file icon in lightbox download file.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T3612: Lightbox v2.
epriestley edited edge metadata.

This probably does the wrong thing if you click a file to bring up the lightbox, then open the comments, then click a file in the file's comments, then click the file icon. But presumably that's rare.

This revision is now accepted and ready to land.Dec 2 2016, 6:12 PM
This revision was automatically updated to reflect the committed changes.

How can I capture/listen to sigils only under a certain node?

To listen only under a particular node:

JX.DOM.listen(root, 'click, 'sigil', function(e) { ... });

To find a node under a particular node:

var thing = JX.DOM.find(root, 'div', 'sigil');

What does JX.Dom.listen vs JX.Stratcom.listen do?

JX.Stratcom.listen() does not take a node, and listens on the whole document.

(Technically, JX.DOM.listen() just puts a unique ID on the node if it doesn't already have one, adds it to all your sigil specifications, then calls JX.Stratcom.listen().)