751677 - Rethink the UI bits for the remote debugging case

Status: RESOLVED WORKSFORME

(DevTools :: Debugger, defect, P3)

Description

[Victor Porof [:vporof][:vp]]

Followup for bug 748927.

Comment 1

[Victor Porof [:vporof][:vp]]

Besides the obvious stuff, we really need an activity indicator, because connections could take a while. Especially when they timeout.

Comment 2

[Rob Campbell [:rc] (:robcee)]

add more throbber?

Comment 3

[Victor Porof [:vporof][:vp]]

(In reply to Victor Porof from comment #1)
> Besides the obvious stuff, we really need an activity indicator, because
> connections could take a while. Especially when they timeout.

By obvious stuff, maybe think of something else than a popup.

Comment 4

[Panos Astithas (he/him) [:past] (please ni?)]

As we discussed with shorlander and paul, the eventual UI for launching remote and chrome debugging should be buttons in the debugger UI and not menu items. That's the main thing for this bug in addition to the activity indicator. A prompt (of some sort) for getting the remote connection endpoint was deemed sufficient, so no need to touch that part for now, I guess.

Comment 5

[Victor Porof [:vporof][:vp]]

Note: for desktop vs. desktop debugging sessions, some UI for starting a server is also needed. Ninjas currently rely on running bits of code found in various mythical gists, which is less than optimal.

Comment 7

[Panos Astithas (he/him) [:past] (please ni?)]

Related bug: bug 802231. Also, we now need to provide an implementation that can be used by all of our remotable tools.

Comment 8

[Victor Porof [:vporof][:vp]]

<past> We also need to add a target dialog to the chrome debugger so that one can direct it to fennec (or thunderbird in the future)

Comment 9

[Victor Porof [:vporof][:vp]]

Looks to me like this bug has been fixed by the Connect screen, apart from the scenario described in comment #5. Filed bug 832163 for that.