Page MenuHomePhabricator

🎬️ desktop-de-07 | Extended banner form | ⏰ Oct 26th
Closed, ResolvedPublic1 Estimated Story Points

Description

After several tests using a second banner form page asking for the address type, we want to test the effect of making this visible initially. The control banner is based on the control banner of desktop-de-05.

Acceptance Criteria

  • Both banners use the control donation form of desktop-de-06.
  • The second banner form page is added to the control banner (as seen in the design file).
  • The variant banner is split into 3 columns (as seen in the design file).
  • The left column containing the text slider has a variable width.
  • The banner's form is never displayed below the text slider.
  • The banners are of the same height.
  • Banner screenshots are uploaded to Shutterbug.

Notes

  • The latest version of the desktop banners should be used.

Event Timeline

kai.nissen set the point value for this task to 8.

Thanks, @AbbanWMDE! Some things, though:

  • The impCount parameter is passed as NaN.
  • The parameter ast=1 should be passed instead of provadd=1.
  • The slide content is left aligned if only one line is used.

I'm also thinking that we should prevent banner display below a viewport width of ~900px. There aren't many impressions, anyway.

kai.nissen changed the point value for this task from 8 to 1.Oct 25 2021, 10:31 AM

@AbbanWMDE
Logging with schema WMDEBannerEvents isn't working well. The close event sends finalSlide as 0 and the events triggered on opening the use of funds and clicking the button at the bottom of the overlay send finalSlide as null.

@kai.nissen Those fixes should be in now, including the finalSlide one

kai.nissen renamed this task from 🎬️ desktop-de-07 | Extended banner form | ⏰ Oct 25th to 🎬️ desktop-de-07 | Extended banner form | ⏰ Oct 26th.Oct 26 2021, 6:02 AM