search

owncloud-archive / documents

ownCloud Documents is collaborative editing of rich-text documents.
http://owncloud.org/
137 stars 55 forks source link

Remove avatars inside of the document #42

Closed MorrisJobke closed 11 years ago

MorrisJobke commented 11 years ago

As a user it's really annoying to have those fancy avatars right inside your text. You want to write and therefore you often need to read the sentence you currently writing. So the avatar is just annoying. I think it enough to have the color <-> user relationship on the left side. Maybe add a on hover effect - if the user hover over a avator in the sidebar the corresponding cursor in the text area gets expanded to the cursor with avatar (like it's now).

cc @jancborchardt

jancborchardt commented 11 years ago

Totally agree, also see https://github.com/owncloud/documents/issues/30 The colored cursor and the sidebar are enough to indicate.

cc @kossebau @VicDeo

adityab commented 11 years ago

There are only a few colors that the human eye can distinguish on a thin vertical line at a glance, so... an alternative suggestion - avoid avatar images over the cursors, but use full name labels instead like in some other collaborative editing apps.

MorrisJobke commented 11 years ago

Most of the time it's equal to me who edit a specific part. I just want to know that someone is in the same area. Therefore I'm fine with just a colored cursor.

jancborchardt commented 11 years ago

Yeah, in this case it’s actually good perception-wise that it’s not too present. Full-name labels would be annoying as well. It’s a collaborative editor and people should be able to focus on what they write, not see a bunch of dancing faces over the text.

VicDeo commented 11 years ago

Is it a duplicate of #30 ?

MorrisJobke commented 11 years ago

@VicDeo Not completely because #30 is just for the avatar of the current user

VicDeo commented 11 years ago

Ok, they are hidden by default

MorrisJobke commented 11 years ago

Thanks

PVince81 commented 10 years ago

An alternative to this could be to make it so that when you click on an avatar, the view jumps to where the cursor of that person is. Hover on an avatar would highlight the cursor. But that would me much more work to implement for probably not much gain.

VicDeo commented 10 years ago

@PVince81 Short story: js editor component is a part of big project developed upstream. It has some patches already but patching should not be a common practice to prevent headaches while downstreaming.

https://github.com/owncloud/documents/tree/master/src/patches