Opened 3 years ago
Closed 2 years ago
#5105 closed change (fixed)
Move right video above content on Acceptable Ads Certification Tool page of acceptableads.com
Reported by: | juliandoucette | Assignee: | |
---|---|---|---|
Priority: | P3 | Milestone: | |
Module: | Websites | Keywords: | |
Cc: | athornburgh, jeen, saroyanm, p.pastourmatzis, jobp, wspee | Blocked By: | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description (last modified by juliandoucette)
Background
The 1/2 column right video on the Acceptable Ads Certification Tool page of acceptableads.com looks a little awkward on tablet and desktop.
What to change
Change the 1/2 columns to 7/12 columns and move the video underneath the heading and hr.
Attachments (6)
Change History (21)
comment:1 follow-up: ↓ 2 Changed 3 years ago by juliandoucette
comment:2 in reply to: ↑ 1 Changed 3 years ago by athornburgh
Replying to juliandoucette:
I can create a mockup if product thinks this is a good idea.
I think it's a good idea and a mockup would be niece as well :)
comment:3 Changed 3 years ago by juliandoucette
This is not looking so hot (especially because of the yellow notice which prevents the text from being seen above the fold). I tried 1/2 column but I thought it looked better as 2/3 column on desktop and tablet. Let me know what you think.
comment:4 Changed 3 years ago by juliandoucette
- Priority changed from Unknown to P4
comment:5 Changed 3 years ago by juliandoucette
- Priority changed from P4 to P5
comment:6 follow-up: ↓ 7 Changed 3 years ago by jeen
I've got some questions/ feedback:
- Why is the Acceptable Ads certification tool (Beta)repeated twice in the breadcrumb? This seems to be a mistake.
- For smaller screens, we should reduce the font size and corresponding padding down. Right now the font size and padding all remain the same. Smaller screens are typically viewed from a shorter distance, so reducing these down won't impair readability.
- The above suggestions aren't unique to this page but I think should be implemented across our site. These updates would also solve the fact that the text goes below the fold on small screens currently.
- I also think that the header shouldn't be sticky, but should appear when the user scrolls down the page.
comment:7 in reply to: ↑ 6 Changed 3 years ago by juliandoucette
Replying to jeen:
I've got some questions/ feedback:
- Why is the Acceptable Ads certification tool (Beta)repeated twice in the breadcrumb? This seems to be a mistake.
Yes. I will create another ticket for this issue.
- For smaller screens, we should reduce the font size and corresponding padding down. Right now the font size and padding all remain the same. Smaller screens are typically viewed from a shorter distance, so reducing these down won't impair readability.
I agree. I will create another ticket for this issue.
- I also think that the header shouldn't be sticky, but should appear when the user scrolls down the page.
This will be addressed by #4868.
Alternatively, we could move the second section underneath the video and keep the 2 column layout. See the following attachment.
comment:8 Changed 3 years ago by juliandoucette
comment:9 follow-up: ↓ 11 Changed 3 years ago by juliandoucette
- Cc jobp added
- Are there any public release or promotion dates for AACT that I should be aware of @jobp?
- Still awaiting your feedback on my latest mock ups @Jeen?
comment:10 Changed 3 years ago by juliandoucette
- Cc wspee added
comment:11 in reply to: ↑ 9 Changed 3 years ago by jeen
Replying to juliandoucette:
- Still awaiting your feedback on my latest mock ups @Jeen?
I wouldn't put the Easy Implementation optimized for your audience text below the video, as it makes it harder for the user to scan the text on the page.
I am in favour of having the video under the title instead.
comment:12 Changed 2 years ago by juliandoucette
- Description modified (diff)
- Priority changed from P5 to P3
- Ready set
- @jlow please confirm priority
- This will basically require a page re-write (function, not content)
comment:13 Changed 2 years ago by jeen
Just rethinking this again:
The video should come as the first item on the page, the title below it, and then the rest of the body text.
This is not a major priority since it is a beta tool, and the video placement does not hinder the functionality of the page.
comment:14 Changed 2 years ago by juliandoucette
- Milestone acceptableads.com/committee cleanup deleted
comment:15 Changed 2 years ago by juliandoucette
- Resolution set to fixed
- Status changed from new to closed
I can create a mockup if product thinks this is a good idea.