Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

www.xnxx.com - The Bookmark button does not trigger the Bookmark prompt #27353

Closed
softvision-sergiulogigan opened this issue Mar 7, 2019 · 5 comments
Labels
browser-firefox engine-gecko The browser uses the Gecko rendering engine nsfw priority-critical q4-2019-outreach Tracking outreach in Q42019. severity-minor The site has a cosmetic issue. type-js Generic JS issues type-uaoverride Require a UA override for working
Milestone

Comments

@softvision-sergiulogigan

URL: https://www.xnxx.com/

Browser / Version: Firefox Nightly 67.0a1 (2019-03-06)
Operating System: Windows 10
Tested Another Browser: Yes

Problem type: Site is not usable
Description: The Bookmark button does not trigger the Bookmark prompt
Steps to Reproduce:

  1. Navigate to: https://www.xnxx.com/
  2. Click on the "Bookmark" button, in th top-right side of the page.

Expected Behavior:
A bookmark prompt is displayed.

Actual Behavior:
Nothing happens.

Notes:

  1. The issue is not reproducible on Chrome.

Watchers:
@softvision-oana-arbuzov
@softvision-sergiulogigan
@cipriansv

sv;

Browser Configuration
  • None

From webcompat.com with ❤️

@webcompat-bot webcompat-bot added this to the needstriage milestone Mar 7, 2019
@softvision-sergiulogigan softvision-sergiulogigan added nsfw severity-minor The site has a cosmetic issue. labels Mar 7, 2019
@softvision-sergiulogigan softvision-sergiulogigan changed the title www.xnxx.com - site is not usable www.xnxx.com - The Bookmark button does not trigger the Bookmark prompt Mar 7, 2019
@softvision-sergiulogigan
Copy link
Author

image

@karlcow karlcow self-assigned this Mar 11, 2019
@karlcow
Copy link
Member

karlcow commented Mar 11, 2019

<a href="#" class="bookmark-link icon-sitenav-bg mobile-hide" target="_self" title="XNXX VIDEOS" rel="sidebar">
  <span class="icon bookmark"></span>
</a>

and this interesting… script.

  define("lib/helpers/bookmark", [], function() {
    return function(e, t, i) {
      (e = $(e)),
        e.data("bookmark-title") && (i = e.data("bookmark-title")),
        e.on("click", function(n) {
          if (window.sidebar && window.sidebar.addPanel)
            window.sidebar.addPanel(t, i, "");
          else if (window.external && "AddFavorite" in window.external)
            window.external.AddFavorite(t, i);
          else {
            if (
              (window.opera && window.print) ||
              (window.sidebar && !(window.sidebar instanceof Node))
            )
              return e.attr("rel", "sidebar").attr("title", i), !0;
            alert(
              "Press " +
                (-1 !== navigator.userAgent.toLowerCase().indexOf("mac")
                  ? "Command/Cmd"
                  : "CTRL") +
                " + D to bookmark this page."
            );
          }
          return n.preventDefault(), !1;
        });
    };
  }),

Basically this is a long list of sniffing attempts.
on macOS Chrome Canary it displays an… alert, just because of "mac" in the ua string.

Moving to needscontact.

@karlcow karlcow added type-js Generic JS issues type-uaoverride Require a UA override for working labels Mar 11, 2019
@karlcow karlcow removed their assignment Mar 11, 2019
@karlcow karlcow modified the milestones: needsdiagnosis, needscontact Mar 11, 2019
@miketaylr miketaylr added the engine-gecko The browser uses the Gecko rendering engine label Apr 30, 2019
@miketaylr
Copy link
Member

              return e.attr("rel", "sidebar").attr("title", i), !0;

We end up in this condition, and support for that stopped working in Firefox in 2018: https://www.fxsitecompat.dev/en-CA/docs/2018/sidebar-link-type-is-no-longer-supported/

@miketaylr
Copy link
Member

Attempting contact via https://info.xnxx.com/contact -- the fix would be to just treat us the same way as Chrome (show the alert).

@miketaylr miketaylr modified the milestones: needscontact, sitewait Oct 4, 2019
@miketaylr miketaylr added the q4-2019-outreach Tracking outreach in Q42019. label Oct 4, 2019
@softvision-oana-arbuzov
Copy link
Member

The issue has been fixed. A popup with info on how to bookmark the page is displayed (Ctrl+D), which activates the "Bookmarks" menu.
image

Tested with:
Browser / Version: Firefox Nightly 102.0a1 (2022-05-04)
Operating System: Windows 10 Pro

[inv_18/2022]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser-firefox engine-gecko The browser uses the Gecko rendering engine nsfw priority-critical q4-2019-outreach Tracking outreach in Q42019. severity-minor The site has a cosmetic issue. type-js Generic JS issues type-uaoverride Require a UA override for working
Projects
None yet
Development

No branches or pull requests

5 participants