-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Share owner avatar position #14457
Comments
Also there's an empty space when sharing with groups, is it ok ? |
@PVince81 mmm on the right makes more sense indeed. I'm not sure why I did not put it there in the first place. Aaah yes the spacing... It indeed looks weird now... It is there so they align better if you have it shared with a user. Should also be fixable. I'll have a look later. |
@rullzer no, the avatar should always be on the left of the name. Here the placeholder of »root« is placed wrongly. And yeah, we should show a group-icon after all for groups. Then it will look less strangely indented when you only shared with a group. |
Ah sorry typo. I meant on the left. |
I have a branch where the avatar is to the left (trivial fix). I'll keep you up to date.. |
@rullzer great stuff, thank you! :) |
We might need an icon/avatar as well for server to server shares (aka ownCloud federation): |
Would be awesome if we can then. :) At least for people whom you already shared with in the past. |
Probably not in this ticket. I raised this one for that: #14564 |
Great stuff, thanks! |
@owncloud/designers @rullzer shouldn't the avatar rather be left or was it decided to have it right on purpose ? Or a bug ?
The text was updated successfully, but these errors were encountered: