[Permission] Geolocalization dialog 'more info' link needs to change to 'hide info'

RESOLVED FIXED

Status

Firefox OS
Gaia::System
--
trivial
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: Adrian Custer, Assigned: chens)

Tracking

unspecified

Firefox Tracking Flags

(b2g-v1.3T affected)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:26.0) Gecko/20100101 Firefox/26.0 (Beta/Release)
Build ID: 20131028225529

Steps to reproduce:

When a packaged app calls 
    navigator.geolocation.getCurrentPosition(...)
FFOS opens a permission dialog which has a 'More info...' link to get details. 

Clicking on the 'More info...' link provides more information but the link name stays the same.

However, since a second click on the link will hide the extra information, the link text should change to something like 'hide info' or 'less info' or some such.

This is happening on Firefox OS 1.1 (Uruguay distributed Alcatel One Touch Fire.) I will check the Simulator after submitting the bug (I need to restart FF for the update to work correctly it seems).


Actual results:

Clicking on the link called 'More info...' showed both more info and a link still called 'More info...'.



Expected results:

After the initial click on the 'More info...' link, the link name should change to a new name such as 'Hide info' or some such.
(Reporter)

Comment 1

5 years ago
Confirmed also on the Firefox 1.2 Simulator run by the App Manager.
(Reporter)

Comment 2

5 years ago
Firefox 1.3 Simulator on the App Manager fails to execute the getCurrentPosition() call; it seems like a no-op so I can not confirm there.

Updated

5 years ago
Component: General → Gaia::System
(Reporter)

Updated

5 years ago
Severity: normal → trivial
(Assignee)

Updated

4 years ago
Assignee: nobody → shchen
(Assignee)

Comment 3

4 years ago
Created attachment 8394589 [details] [review]
Pull request
Attachment #8394589 - Flags: review?(alive)
Comment on attachment 8394589 [details] [review]
Pull request

Hi, please try to have a test for the functionality you added. Thanks.
Attachment #8394589 - Flags: review?(alive)
(Assignee)

Updated

4 years ago
Attachment #8394589 - Flags: review?(alive)
Attachment #8394589 - Flags: review?(alive) → review+
(Assignee)

Comment 5

4 years ago
Merged to master, r=alive
https://github.com/mozilla-b2g/gaia/commit/18a0615e4aa939d8ed4504ca1463916e67e619a4
Status: UNCONFIRMED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Seeing this issue on Tarako v1.3

1.3 Environmental Variables:
Device: Tarako 1.3
BuildID: 20140415004002
Gaia: 44ff6248c28ff83b9ad1161847a176399f93d3bb
Gecko: 28aea220e338
Version: 28.1
Firmware Version: SP6821
status-b2g-v1.3T: --- → affected
You need to log in before you can comment on or make changes to this bug.