Page MenuHomePhabricator

[Bug][Regression] Tapping "Refresh location" in Nearby goes to search
Closed, ResolvedPublic


Steps to reproduce

  1. Visit
  2. Tap the "refresh location" button

Expected results

Results are updated using the user's current geolocation.

Actual results

An empty search page is shown (

Environments observed

Browser Version:

  • Chromium v63.0.3239.132 (Official Build) Built on Ubuntu , running on Ubuntu 17.10 (64-bit)

OS Version:

  • Ubuntu v17.10 64b

Device Model:

  • Desktop

Device Language:

  • English

Testing criteria

Event Timeline

Jdlrobson renamed this task from [Bug] Tapping "Refresh location" in Nearby goes to search to [Bug][Regression] Tapping "Refresh location" in Nearby goes to search.Feb 1 2018, 6:19 PM
Jdlrobson moved this task from Incoming to Triaged but Future on the Readers-Web-Backlog board.
Jdlrobson added subscribers: ovasileva, Jdlrobson.

Oops. @ovasileva high or normal priority?

xSavitar triaged this task as High priority.
xSavitar awarded a token.
xSavitar added a project: MobileFrontend.
xSavitar added a subscriber: xSavitar.

Change 407477 had a related patch set uploaded (by D3r1ck01; owner: Alangi Derick):
[mediawiki/extensions/MobileFrontend@master] Fix refresh button on Special:Nearby page

@Jdlrobson, I remember we worked on a similar issue sometime ago? How come it has resurfaced? :) I've just checked and this is it, But it's similar :)

xSavitar moved this task from Backlog to Team: web on the MobileFrontend board.

@Jdlrobson, the main issue for adding some css was to remove this effect on the button on-click;

Note the blue shadow around the button? But if this is not needed, I can remove it. Thanks :)

Oops. @ovasileva high or normal priority?

High works.

Jdlrobson added subscribers: Volker_E, Nirzar.

The main issue of the submitting search has been fixed by a volunteer.
@Volker_E and @Nirzar can you take a look and maybe open a follow up if we need to change anything with regards to the blue shadow per ? Thanks!

@Jdlrobson If I understand correctly, the refresh button pointed at the wrong action. This is fixed now.
Having a focus feedback is an essential from accessibility perspective. Why is the focus outline seen as problematic?

blue shadow is the focus ring, we need it. not problematic from design POV

Thanks all. Looks like some follow up might be needed anyhow.

@Jdlrobson, following @matmarex's comment on Gerrit, I've updated the patch to use <button type='button'>...</button> which also solves the issue of the blue shadow around the button when clicked. But the only issue now is Jenkins is reporting a failing build but this doesn't relate to the patch AFAIK. Any ideas?

Jenkins is now happy with the patch all of a sudden :) No need to worry again, @matmarex can confirm this!

Change 407477 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Fix refresh button on Special:Nearby page

Jdlrobson updated the task description. (Show Details)

Over to you @ABorbaWMF - this will go out on the train tomorrow.

Sorry for the delay. I thought the train had not yet arrived because I was getting no results at all on nearby on Beta, but then realized there are no articles tagged with Alameda. This is working now on production.