Create a modal for entering a JSON object that is merged with the auto-generated package.json

RESOLVED FIXED in Q1 2012

Status

addons.mozilla.org Graveyard
Add-on Builder
P1
enhancement
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: dbuc, Assigned: seanmonstar)

Tracking

unspecified
Q1 2012

Details

(Reporter)

Description

6 years ago
We need a modal UI that allows developers to input a custom JSON object that is merged into the auto-generated package.json at during the package create cycle. This will allow for developers to add icons, preferences, and localization to their add-ons.

The UI should consist of an ACE editor and a save button. Before save, we will ensure that the text the user is saving actually generates an object. Once saved, this JSON object will have the auto-generated JSON object that makes up the package.json merged *into* it. We want to merge into it to ensure that any critical, auto-generated values are not overwritten by the user in their custom object.
(Reporter)

Updated

6 years ago
Assignee: nobody → smcarthur
Severity: major → enhancement
Priority: -- → P1
Target Milestone: --- → Q1 2012
(Reporter)

Updated

6 years ago
Blocks: 715698
(Assignee)

Comment 1

6 years ago
I'm considering adding an extra text field (turned into an ACE) on the current Properties modal, since well, it's extra properties.
(Reporter)

Comment 2

6 years ago
Sounds good to me, just make sure to validate the text from that ACE editor area before posting that form ;)
(Reporter)

Comment 3

6 years ago
Here is some CSS mods to make the editor area in the modal look better: 

#extra_json_ace {
	margin-right: -2px;
	border: 1px solid #aaa;
}

#extra_json_ace div.ace_scroller {
	border: none;
	border-radius: 0;
	box-shadow: 1px 1px 2px -2px #000; 
}

#extra_json_ace div.ace_sb {
	border-top: none;
	border-left: 1px solid #bbb;
}
(Assignee)

Comment 4

6 years ago
Doneskis.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.