28 Jan
2010
28 Jan
'10
12:08 a.m.
Hi Sergiu,
The design is good overall. Here are some comments:
* the information is overcrowded; I would add more space between the
elements and make the labels less striking
* uppercase is fine for one word but otherwise makes the text hard to
read; (why are the labels shouting at me?)
* it's not clear how multiple includes are displayed
* any reason (other than "there's no room for it") for not showing the
list of included documents in WYSIWYG mode?
* what happens with the syntax help if I don't select an XWiki syntax?
Sergiu Dumitriu wrote:
Hi devs,
We've been working on improving the editors (content, class, object),
and now I have some pretty important UI changes to commit, but not
everybody seems to agree with them, so I bring up a vote.
The whole improvements can be seen on
http://incubator.myxwiki.org/xwiki/bin/Improvements/ImprovedEdit
Here are the individual voted points:
0 Remove all panels in edit mode
If your goal is in-line/in-place editing why removing the view panels?
1a Parent and title above the content in wiki/wysiwyg mode
+1
1b The same style in edit mode as in view mode for the parent/title fields
-0
1c AJAX Suggest for the parent field
+1
2a New label for the content textarea ("Content")
+0
2b List of included documents after the Content label
I'm not convinced it scales. I'd like to see the list in WYSIWYG mode also.
3a Syntax switcher in the top right corner of the content
+1
3b Syntax help in the top right corner of the content
+0
3c Syntax help and switcher only in the wiki editor
-0
4 Better label for the version comment
+1 but please don't display it all in uppercase
5a Right float the Minor edit field
+0
5b Put the Minor edit label after the checkbox
+1
6 Move autosave in line with the submit buttons
+1
7 Introduce new AJAX-powered Add Object
i) above the objects
ii) bellow the objects
+1 for any of them
8 Introduce new AJAX-powered Add Object from this class
i) at the top of the list of existing objects
ii) at the end of the list
+1 for any of them
9 Move the class switcher in the top right corner
+0
9b Remove the class switcher
+0 it doesn't belong there but at the same it can be very useful
10 Introduce new AJAX-powered Add Property
i) at the top of the list of properties
ii) at the end of the list
+1 for any of them
Thanks,
Marius
I vote +1 for all of the above, except 9b (-0), and with options 7i),
8ii), and 10ii)