Nice job! my remarks: - I think also the "recent changes" section is a bit "oversized" compared to the others sections because when you open the page, you only see this at first sight. - The number of items in recent changes should be reduced (it is already possible as you said in a previous mail) so that the recent changes is not too tall compared to Spaces+Tags and so that you can guess there are other contents below recent changes without scrolling too much. - Maybe default height for one recent change could be reduced a bit by putting the name just beside the photo for example... I like this size of photo... - considering balance, I would prefer a smaller column for recent changes... not 50/50 but 60/40 for example... this is a personal opinion :) - I agree the avatar presentation brings something interesting.
Pascal On Tue, Jan 20, 2009 at 9:38 PM, Marta Girdea <[email protected]>wrote: > On Tue, Jan 20, 2009 at 9:19 PM, Jerome Velociter <[email protected]> > wrote: > > > Jean-Vincent Drean wrote: > > > On Tue, Jan 20, 2009 at 6:52 PM, Marta Girdea <[email protected]> > > wrote: > > >> This is starting to look really nice, I particularly like the fact > that > > the > > >> recent changes are displayed in a more comprehensive manner. > > >> > > >> I have a few comments though: > > >> 1/ The two columns look a bit unbalanced. Isn't there some other > > important > > >> information the user would like to see on in the dashboard, that could > > fill > > >> the empty space? Or maybe we could reduce the number of displayed > recent > > >> changes? > > > > > > I think that the average wiki contains more spaces than incubator (way > > > more in some cases). > > > You can try to reduce the number of items in recent changes by passing > > > changesNb=15 in the URL (this variable can be set in the document > > > too). > > > > > >> 2/ There are some scalability problems: on the 'Recent changes' > column, > > if > > >> the document name is long, 'see modifications' that appears on hover > > shows > > >> up on the next row and pushes the rest of the content down, and this > is > > not > > >> pretty. What could be done is use an icon instead of text, and this > way > > we'd > > >> have consistency with the 'Spaces' section. > > > > > > It's not pretty but I've asked for feedback during the devel process > > > and since this onhover link vary a lot (link vs. ajax) between items > > > (page vs. comment) it seemed to be way more comprehensive to write it > > > and use visual hints (color + [+] and [-]) than to use images. > > > > > >> 3/ I don't think the user's avatar is a crucial information to display > > in > > >> Recent changes. Also, too few or too many document changes per user > will > > >> result in empty spaces. Maybe reducing the image's size would partly > > solve > > >> the problem. > > > > > > This is the first negative feedback about avatars, I agree it's not > > > crucial but it allow to put a face on changes which is a good thing I > > > think. > > > > Yes, power to people! Avatars makes a big difference in the tool > > adoption by its users, in my opinion. (And displaying it other places > > than user own profile encourages people to actually upload their > picture). > > > I agree, that's why I didn't suggest removing the avatars, just making them > smaller. This wasn't negative feedback on the avatars, but a comment on the > fact that they are way too visible comparing to the rest (at least on my > display). > > > Jerome. > > > > > I'm afraid that a lot of avatars will not look pretty with a height > > > smaller than 60px. > > > > > > Thanks, > > > JV. > > > _______________________________________________ > > > devs mailing list > > > [email protected] > > > http://lists.xwiki.org/mailman/listinfo/devs > > > > _______________________________________________ > > devs mailing list > > [email protected] > > http://lists.xwiki.org/mailman/listinfo/devs > > > _______________________________________________ > devs mailing list > [email protected] > http://lists.xwiki.org/mailman/listinfo/devs > _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs

