Page MenuHomePhabricator

Set max width container for banners
Closed, ResolvedPublic2 Estimated Story Points

Description

Set max width container for banners to 893 px.
Run it by fundraising team for sanity check.

How to test with ZeroBanner extension:

  1. Install ZeroBanner extension on local instance.
  2. Add a fake partner config by creating an article within the Zero namespace (I recommend "Zero:000-00"). There should be a default configuration, but here's an example: http://pastebin.com/QaA0TeE2. Note that the "enabled" field must be set to true and there must be a banner message.
  3. Add a request header (I recommend installing a browser extension for modifying headers such as "Modify Headers") with key-value pair "X-CS, <config-name>" (e.g. X-CS, 000-00).
  4. Navigate to local instance of Main Page article.

If this set of steps does not work, feel free to ping me (jhobs) on gchat or IRC.

Event Timeline

KHammerstein raised the priority of this task from to Needs Triage.
KHammerstein updated the task description. (Show Details)
KHammerstein subscribed.
kaldari set Security to None.

Need to check with Partnership team on whether this has implications for ZeroBanner extension that need to be cleared with mobile partners first.

Change 210616 had a related patch set uploaded (by Bmansurov):
Make banners center-aligned similar to the content

https://gerrit.wikimedia.org/r/210616

@jhobs, thanks for clear instructions. It was easy to set the testing environment up. By the way, do you know who from the Partnership and Fundraising teams I should ping on here?

@kaldari, any idea who from Partnership/Fundraising I should ping to get this reviewed?

@jhobs: As @bmansurov said, thanks for the clear instructions on setting up a test environment.

Change 210616 merged by jenkins-bot:
Alpha, Beta: Make banners center-aligned

https://gerrit.wikimedia.org/r/210616