Closed Bug 1137422 Opened 9 years ago Closed 9 years ago

rename project in WebIDE does not work

Categories

(DevTools Graveyard :: WebIDE, defect)

39 Branch
x86_64
All
defect
Not set
normal

Tracking

(firefox40 verified)

VERIFIED FIXED
Firefox 40
Tracking Status
firefox40 --- verified

People

(Reporter: abdelrahman, Assigned: abdelrahman)

Details

(Whiteboard: [bugday-20150708])

Attachments

(1 file)

Steps to reproduce:
1. Open WebIDE.
2. Create new packaged app and use the Hello world template.
3. Right-click on project name in left and choose Rename.
4. Rename it to "test".
Actual result: The project name does not change.
I think we need to hide rename from context menu for the root
for two reasons:
1. conflicts with current code in rename.js when getting its parent.
2. Does not rename the project name, It's renaming the root path.

Project can be renamed through |manifest.webapp|
like description and launch_path can be edited there.
Assignee: nobody → a.ahmed1026
Status: NEW → ASSIGNED
Flags: needinfo?(jryans)
I think it makes sense to hide it, yes.

I will be out next week, so it may take me some time to respond.
Flags: needinfo?(jryans)
Attachment #8570802 - Flags: review?(jryans)
Comment on attachment 8570802 [details] [diff] [review]
rev 1 - hide rename for root

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

Great, this looks good to me.  Sorry about the delay!
Attachment #8570802 - Flags: review?(jryans) → review+
Keywords: checkin-needed
Ah, thanks for following up here!

But the sheriffs who land our patches will want a try run before landing it.  Let me do that now.
Keywords: checkin-needed
(In reply to J. Ryan Stinnett [:jryans] from comment #5)
> But the sheriffs who land our patches will want a try run before landing it.
> Let me do that now.

Thanks :)
https://hg.mozilla.org/mozilla-central/rev/0a899ed1487b
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 40
I found the bug on Firefox nightly windows 32 bit-

Build ID 	20150226030225
User Agent 	Mozilla/5.0 (Windows NT 6.3; rv:39.0) Gecko/20100101 Firefox/39.0


And this looks fixed on :

Build ID 	20150708030204
User Agent 	Mozilla/5.0 (Windows NT 6.3; rv:42.0) Gecko/20100101 Firefox/42.0

[bugday-20150708]
QA Whiteboard: [bugday-20150708]
Whiteboard: [bugday-20150708]
What I have understand is, App rename feature is hidden. 

Successfully reproduce the bug on Firefox 39 (20150630154324).

The fix works for me on Firefox 40.0 (Build ID: 20150706172413, User Agent Mozilla/5.0 (X11; Linux i686; rv:40.0) Gecko/20100101 Firefox/40.0) 

Based on the Comment 10 and my verification I am marking the bug as verified.
Status: RESOLVED → VERIFIED
Product: Firefox → DevTools
Product: DevTools → DevTools Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: