Donation Prompt Blocks Interaction with Bottom of Pages After Being Dismissed #104653

Closed
opened 2023-12-07 08:07:13 +01:00 by MagnusVesper · 2 comments

Page Information
File: fund.blender.org/static/js/donation-box.js (not in this repo)
Blender Version: 4.0
Documentation Language: en
Permanent Link

Short description of error
The donation prompt will continue to block interaction with the page after it has been dismissed. This happens even in new tabs that remember you hiding it previously.
It prevents viewers from clicking the "Next" and "Previous" links on most manual pages along with all links on the blender.org footer. So I suspect many people think the websites are broken from this.

It could be fixed by adding 2 lines to the donationBoxMessageShow function defined in fund.blender.org/static/js/donation-box.js:

function donationBoxMessageShow() {
  if (donationBoxMessage == null) {
  } else {
    if (checkCookie('donationBoxMessageIsDismissed') || checkCookie('donatedAt')) {
      donationBoxMessage.classList.remove('show');
/*+*/ donationBoxMessage.classList.add('d-none'); /* added */
    } else {
      donationBoxMessage.classList.add('show');
/*+*/ donationBoxMessage.classList.remove('d-none'); /* added */
    }
  }
}

That function is run after clicking "I already donated", "Remind me later", and on page load.
d-none is defined with

.dfdb .d-none {
  display:none !important
}

in inline css included in the donationBoxMessage's parent. I checked this on blender.org and docs.blender.org

I'm not sure the proper place to submit this report. The source to this is not in the blender-manual repo. The problem also exists on blender.org. But this is the only public repo that I could find having anything to do with a website.

**Page Information** File: `fund.blender.org/static/js/donation-box.js` (not in this repo) Blender Version: `4.0` Documentation Language: `en` [Permanent Link](https://fund.blender.org/static/js/donation-box.js) **Short description of error** The donation prompt will continue to block interaction with the page after it has been dismissed. This happens even in new tabs that remember you hiding it previously. It prevents viewers from clicking the "Next" and "Previous" links on most manual pages along with all links on the blender.org footer. So I suspect many people think the websites are broken from this. It could be fixed by adding 2 lines to the `donationBoxMessageShow` function defined in `fund.blender.org/static/js/donation-box.js`: ```js function donationBoxMessageShow() { if (donationBoxMessage == null) { } else { if (checkCookie('donationBoxMessageIsDismissed') || checkCookie('donatedAt')) { donationBoxMessage.classList.remove('show'); /*+*/ donationBoxMessage.classList.add('d-none'); /* added */ } else { donationBoxMessage.classList.add('show'); /*+*/ donationBoxMessage.classList.remove('d-none'); /* added */ } } } ``` That function is run after clicking "I already donated", "Remind me later", and on page load. `d-none` is defined with ```css .dfdb .d-none { display:none !important } ``` in inline css included in the `donationBoxMessage`'s parent. I checked this on blender.org and docs.blender.org I'm not sure the proper place to submit this report. The source to this is not in the blender-manual repo. The problem also exists on blender.org. But this is the only public repo that I could find having anything to do with a website.
MagnusVesper added the
Status
Needs Triage
Priority
Normal
Type
Report
labels 2023-12-07 08:07:14 +01:00
Thomas Dinges added
Status
Confirmed
and removed
Status
Needs Triage
labels 2023-12-07 08:29:24 +01:00

Thanks for the report and the suggested fix. This will be looked at shortly.

Thanks for the report and the suggested fix. This will be looked at shortly.

This has been fixed and rolled out. Due to caching it might still take a bit until resolved for everyone.

This has been fixed and rolled out. Due to caching it might still take a bit until resolved for everyone.
Blender Bot added
Status
Archived
and removed
Status
Confirmed
labels 2023-12-07 09:21:27 +01:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-manual#104653
No description provided.