fix design details in comments sidebar

[jancborchardt]

Currently there’s a textarea for comment posting in the sidebar. I’d say we only use an input field to encourage short and concise comments.
When we use comments in the Activity app etc, we can use textareas, but not in smaller areas like the sidebar.
Then also pressing enter now sends the message, much quicker.

And the layout is improved overall. Focus on the message rather than author and date, which only get half opacity.

Before & after:

Please review @nextcloud/designers :)

[mention-bot]

@jancborchardt, thanks for your PR! By analyzing the annotation information on this pull request, we identified @PVince81, @vincchan and @MorrisJobke to be potential reviewers

[vincchan]

double quotes ending

[vincchan]

looks really nice and works! :) 👍

[vincchan]

failing test

PhantomJS 2.1.1 (Linux 0.0.0) OCA.Comments.CommentsTabView tests editing comments saves message and updates comment item when clicking save FAILED
Expected Object({ message: 'New message' }) to equal Object({ message: 'modified
message' }).
/drone/src/github.com/nextcloud/server/apps/comments/tests/js/commentstabviewSpec.js:335:45

EDIT: also when we're on the subject

editing comments looks a bit weird atm compared to newcomments form

[williambargent]

Looks much nicer, Works
👍

[MorrisJobke]

• Editing the comment is broken:

[MorrisJobke]

editing comments looks a bit weird atm compared to newcomments form

-.- Oh ... it was already reported.

@jancborchardt Please fix this ;)

[MorrisJobke]

Beside that it looks nice :)

[rullzer]

• Fixed css
• fixed js test

Only because it is an input now so you can now longer format text (newlines).
Which might get messed up if you want to edit a comment from a different place that does contain newlines.

I don't really have an opinion here. But let you all decide.

Review time

[MorrisJobke]

I fixed the layout of the submit button and tested it in IE10, 11, Edge, Chrome and Firefox. It now also works properly for all screen sizes.

Before & After:

[MorrisJobke]

I also fixed the layout of the cancel button (pulled to right and properly aligned). I tested this in Chrome, Firefox, Safari, IE10 and Edge.

Before and after:

[MorrisJobke]

I tested it and it now works really great 👍 from me

[rullzer]

LGTM!

[nickvergessen]

This still had open comments...

Will fix #714 (comment)

[nickvergessen]

Fix in #928