owncloud-archive / chat

GNU Affero General Public License v3.0
38 stars 21 forks source link

Specific interface enhancements #20

Closed jancborchardt closed 10 years ago

jancborchardt commented 10 years ago

Moved the checklist from #14 to here. Based on the screenshot of the current state which @LEDfan posted there.

jancborchardt commented 10 years ago

@jbtbnl @stefan-niedermann @LEDfan btw you should join the #owncloud-design and #owncloud-dev channels on IRC (freenode) if you are not on there yet. :)

LEDfan commented 10 years ago
jancborchardt commented 10 years ago

Some more, on the initial launch experience:

@jbtbnl if you’re looking for a new design project to tackle after the redesign of contacts, this seems very fitting. Especially because it’s so close to Contacts and needs tight integration with it. :)

LEDfan commented 10 years ago

@jancborchardt I'll implement it :) But some of them can only be implemented after the contacts is integrated.

Just a question. Why is it bad to use alert boxes? Isn't is good for mobile devices?

jancborchardt commented 10 years ago

@LEDfan because alert boxes:

:)

LEDfan commented 10 years ago

Okay thanks for you response.

Should there be an overlay over the chat messages? Or where should the input field be located?

jancborchardt commented 10 years ago

In the area where the chat is to be (app-content), on the top. Because that needs to be empty.

Hm, that just reminds me, we also need a mechanism for when you add a person to an existing conversation.

So it should be similar to this Bootstrap Popover: http://getbootstrap.com/javascript/#popovers (maybe you can just use that JS library) Extend to the right of the entry / new conversation button, with an input field.

For a completely new Conversation, in the future we should have something like I described above, showing favorited contacts. See Google Hangouts or the Dialer app on Android for that.

LEDfan commented 10 years ago

@jancborchardt I removed the prompt to start a new conversation. Can you have a look at it? I have just copied the css and the generated HTML from bootsrap. Because it gave a lot of troubles when I use Angularstrap or angular-ui-bootstrap. (there couldn't be a form in the popover etc)

LEDfan commented 10 years ago

I implemented the comments above in the code. On startup: oc-chat-1

Popover to start a conversation: oc-chat-2

Conversation with messages and inviting a user: oc-chat-3

Chatting mobile portrait screenshot_2014-02-02-11-35-20

Chatting mobile landscape screenshot_2014-02-02-11-33-41

Popover Mobile Landscape screenshot_2014-02-02-11-33-56

More suggestions?

jancborchardt commented 10 years ago

Nice progress! @LEDfan these comments are purely from looking at the screenshots because currently I have no time to look at it. Hope it’s helpful. :)

On startup

Popover to start a conversation

Popover Mobile Landscape

@jbtbnl do you also have remarks?

jbtbnl commented 10 years ago

Some remarks from my side:

jancborchardt commented 10 years ago

@jbtbnl that’s what I meant by moving away from the popover – thanks for stating an example! :) Also agree with your second point.

Another thing:

LEDfan commented 10 years ago

I'll update the code ASAP :) But for the name in the conversation list. Should there be a vertically list with users like so:

user1avatar user1name
user2avatar user2name 

etc or: user1avatar user2avatar user1name user2name

jancborchardt commented 10 years ago

@LEDfan as soon as multiple users are in the conversation, just don’t show their names anymore.

LEDfan commented 10 years ago

Okay I'll do that. Thanks.

LEDfan commented 10 years ago
jancborchardt commented 10 years ago
LEDfan commented 10 years ago

@jancborchardt I have used the plugin you mentioned. What should be the behaviour when enter is pressed inside the texarea:

stefan-niedermann commented 10 years ago

most messenger allow configuration but use the first version (enter = send, shift + enter = new line) as default. a message should be sent on pressing enter (and shift + enter = new line). imho we don't need to allow configuration here (or maybe a configuration file? - nothing in the user interface).

jbtbnl commented 10 years ago

Yes, enter = send, shift + enter = new line as suggested by @stefan-niedermann is the most common.

jancborchardt commented 10 years ago

Yep. No configuration needed.

@stefan-niedermann btw you should join our #owncloud-design (and #owncloud-dev) IRC channels! (on freenode)

stefan-niedermann commented 10 years ago

@jancborchardt how often do you want to repeat this? :D :D :D

I'm already at the mailing list. IRC... well, noone i know uses it, so i use it very rarely myself ;) sometimes i'm in the IRC channels, but than nobody seems to be online or there is currently no discussion ;)

for example like now, or am i doing something wrong?

LEDfan commented 10 years ago
LEDfan commented 10 years ago

I fixed all todos in here. Please open a new issue with new design suggestions/