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

