iLoveElysia/pjax
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 9 Wiki Activity

Add blur on active element when page is changed #4

New Issue
Closed
opened 2014-04-22 00:01:12 -05:00 by MoOx · 4 comments
MoOx commented 2014-04-22 00:01:12 -05:00 (Migrated from github.com)
Copy Link

To remove focus on a clicked/tapped link (eg on ipad, a attached tooltip can remain visible after the new page load, which is weird)

To remove focus on a clicked/tapped link (eg on ipad, a attached tooltip can remain visible after the new page load, which is weird)
MoOx commented 2014-04-23 08:26:54 -05:00 (Migrated from github.com)
Copy Link

Be careful to this issue https://github.com/defunkt/jquery-pjax/issues/377

Be careful to this issue https://github.com/defunkt/jquery-pjax/issues/377
BehindTheMath commented 2018-01-23 23:55:18 -05:00 (Migrated from github.com)
Copy Link

Add blur on active element when page is changed
To remove focus on a clicked/tapped link (eg on ipad, a attached tooltip can remain visible after the new page load, which is weird)

@MoOx Isn't this covered by this line and this line? I don't know why both are needed, but regardless, the focus should be lost.

Be careful to this issue defunkt/jquery-pjax#377

This should be a simple fix, similar to what was ultimately implemented here.

>Add blur on active element when page is changed >To remove focus on a clicked/tapped link (eg on ipad, a attached tooltip can remain visible after the new page load, which is weird) @MoOx Isn't this covered by [this line](https://github.com/MoOx/pjax/blob/3c8cd087c5352b92fb7f1090b0c39b29cfa7a31a/index.js#L132) and [this line](https://github.com/MoOx/pjax/blob/3c8cd087c5352b92fb7f1090b0c39b29cfa7a31a/index.js#L132)? I don't know why both are needed, but regardless, the focus should be lost. >Be careful to this issue defunkt/jquery-pjax#377 This should be a simple fix, similar to what was ultimately implemented [here](https://github.com/defunkt/jquery-pjax/pull/545).
MoOx commented 2018-01-24 03:26:18 -05:00 (Migrated from github.com)
Copy Link

You pointed the same line, but I guess you are right. Maybe I was referring to event triggered by js (eg: tooltip opened on a tap/click that should be hidden when tapping outside), can't remember...

You pointed the same line, but I guess you are right. Maybe I was referring to event triggered by js (eg: tooltip opened on a tap/click that should be hidden when tapping outside), can't remember...
BehindTheMath commented 2018-01-24 08:45:14 -05:00 (Migrated from github.com)
Copy Link

Sorry. I meant this line and this line.

Sorry. I meant [this line](https://github.com/MoOx/pjax/blob/3c8cd087c5352b92fb7f1090b0c39b29cfa7a31a/index.js#L132) and [this line](https://github.com/MoOx/pjax/blob/3c8cd087c5352b92fb7f1090b0c39b29cfa7a31a/index.js#177).
👍 1
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
dependencies
Pull requests that update a dependency file
 duplicate
enhancement
invalid
question
wontfix
No Label bug
Milestone
No items
No Milestone 0.2.5
Projects
Clear projects
No project
Assignees
Clear assignees
iLoveElysia
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: iLoveElysia/pjax#4
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?