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)
> 
_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs

Reply via email to