1055244 - Re-enable "app manager" menu item

Status: RESOLVED WONTFIX

(DevTools Graveyard :: WebIDE, defect)

Description

[Paul Rouget [:paul]]

Change is too brutal for some develoeprs. Doesn't hurt to keep the app manager menuitem.

Comment 1

[Paul Rouget [:paul]]

Attachment: v1

Comment 2

[Paul Rouget [:paul]]

Comment on attachment 8474767 [details] [diff] [review]
v1

What do you think? Let's keep them both for some time... (not sure how long).

Comment 3

[Kevin Grandon :kgrandon]

Just making a note that app manager has been constantly crashing for me recently. Not sure if it's related to WebIDE being enabled, but if so maybe it'd do more harm to re-enable it than completely disable it.

Comment 4

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Let's discuss this at tomorrow's meeting.

Comment 5

[Paul Rouget [:paul]]

(In reply to Kevin Grandon :kgrandon from comment #3)
> Just making a note that app manager has been constantly crashing for me
> recently.

What do you mean by crashing? Real crash? Did you have the same symptom with WebIDE?

Comment 6

[Kevin Grandon :kgrandon]

(In reply to Paul Rouget [:paul] (slow to respond. Ping me on IRC) from comment #5)
> What do you mean by crashing? Real crash? Did you have the same symptom with
> WebIDE?

Ah, actually it appears it's only because I use ctrl+F to find the app usually :) Maybe not an issue with the app manager, just my workflow. Sorry about the noise.

https://crash-stats.mozilla.com/report/index/6e5827da-843a-4c33-b224-707582140819

Comment 7

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8474767 [details] [diff] [review]
v1

Review of attachment 8474767 [details] [diff] [review]:
-----------------------------------------------------------------

The patch is fine.  But let's hold off landing this until we're sure it's needed.

I'd prefer that people really do move to WebIDE.  Restoring the menu item seems like going backwards to me.

Comment 8

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

No longer interested in doing this.  Hoping to remove AM entirely soon.