Page MenuHomePhabricator

On mobile, the Flow notification's link takes you to the desktop version of the Flow page, even though the main (background) link takes you to the mobile one
Closed, DuplicatePublic

Description

Tested:

  • Topic Foo responded on Pagename -- ("Topic" goes to desktop, background to mobile) <- bug
  • Topic Foo created a new topic on Pagename -- ("Topic" goes to desktop, background to mobile) <- bug
  • Foo mentioned you in their [post] in "Topic" on "Pagename" -- ([post] goes to desktop, background to mobile) <- bug
  • Foo thanked you for [your comment] in "Topic" on Pagename -- (only a background link which goes to mobile) <- success

Event Timeline

Jdforrester-WMF raised the priority of this task from to Needs Triage.
Jdforrester-WMF updated the task description. (Show Details)
Jdforrester-WMF added a subscriber: Jdforrester-WMF.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 18 2015, 1:03 AM

Which notification was it (new topic, thanks, etc.)?

Mattflaschen-WMF triaged this task as Normal priority.Mar 18 2015, 5:38 PM
Quiddity updated the task description. (Show Details)Mar 19 2015, 2:15 AM
Quiddity set Security to None.
Quiddity added a subscriber: Quiddity.

I've added a description, with details

we should be able to write a test that renders all the possible notifications in both mobile and desktop, and then looks at all the links in the final html

Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptApr 10 2015, 5:29 PM