Doug Turner (:dougt) Assignee
	Description • 16 years ago

Recently 459893 change the way geolocation prompts worked.  We need to update the geolocation prompt work in fennec.

	Madhava Enros [:madhava]
	Comment 2 • 16 years ago

Interaction flow for two cases -- when the device (mobile device, laptop, desktop computer) has some ability to know its location (whether by gps, cell-tower association, or known wifi locations), and one where the device just can't know (most likely in the case of a wired-ethernet desktop PC.  The latter case assumes that there's still a network connection, which seems reasonable, given that the user is doing something with the web.  We need the network connection so that fennec/firefox can turn the user-entered info into coordinates.

	Doug Turner (:dougt) Assignee
	Comment 3 • 16 years ago

madhava, do you have any new mockups based on our last discussion?

	Doug Turner (:dougt) Assignee
	Comment 5 • 16 years ago

actual patch to mobile-browser

	Mark Finkle (:mfinkle) (use needinfo?)
	Comment 6 • 16 years ago

Comment on attachment 362952 [details] [diff] [review]
patch v.1

>diff --git a/components/Makefile.in b/components/Makefile.in

> EXTRA_COMPONENTS = \
>     aboutFirstrun.js \
>+    geolocation-prompt.js \

Can we rename the file to geolocationPrompt.js (yes, I'm anal)

>diff --git a/components/geolocation-prompt.js b/components/geolocation-prompt.js

>+        label: browserBundle.GetStringFromName("geolocation.exactLocation"),
>+        accessKey: browserBundle.GetStringFromName("geolocation.exactLocationKey"),
>+        callback: function() request.allow() ,

Remove trailing comma

>+        label: browserBundle.GetStringFromName("geolocation.nothingLocation"),
>+        accessKey: browserBundle.GetStringFromName("geolocation.nothingLocationKey"),
>+        callback: function() request.cancel() ,

Remove trailing comma

>+      notificationBox.appendNotification(message,
>+                                         "geolocation",
>+                                         "chrome://browser/skin/Info.png",
>+                                         notificationBox.PRIORITY_INFO_HIGH,
>+                                         buttons);
>+    }
>+  },

Remove trailing comma

	Doug Turner (:dougt) Assignee
	Comment 7 • 16 years ago

patch v.1 was checked in:  a9713858b4ec

This provides us with a simple UI - similar to firefox - to manage geolocation requests.

	Doug Turner (:dougt) Assignee
	Comment 8 • 16 years ago

this is fixed.  new design of this prompt should be done in bug 463235 per madhava.

	Doug Turner (:dougt) Assignee
	Comment 9 • 16 years ago

Comment on attachment 362951 [details]
basic geolocation prompt (similar to FF)

clearing review flag

	Joel Maher ( :jmaher ) (UTC -8)
	Comment 10 • 15 years ago

verified with beta3

	Aakash Desai [:aakashd]
	Comment 11 • 15 years ago

We have these testcases located in litmus. Ids are: 7651-7654