Skip to content
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

Closed
PVince81 opened this issue Feb 24, 2015 · 11 comments
Closed

Share owner avatar position #14457

PVince81 opened this issue Feb 24, 2015 · 11 comments

Comments

@PVince81
Copy link
Contributor

@owncloud/designers @rullzer shouldn't the avatar rather be left or was it decided to have it right on purpose ? Or a bug ?

share_owner_avatar

@PVince81
Copy link
Contributor Author

Also there's an empty space when sharing with groups, is it ok ?

@rullzer
Copy link
Contributor

rullzer commented Feb 24, 2015

@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.

@jancborchardt
Copy link
Member

@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.

@rullzer
Copy link
Contributor

rullzer commented Feb 24, 2015

Ah sorry typo. I meant on the left.

@rullzer
Copy link
Contributor

rullzer commented Feb 24, 2015

I have a branch where the avatar is to the left (trivial fix).
I still need to look into the avatars for groups and remote users... since there are a lot of edge cases.

I'll keep you up to date..

@jancborchardt
Copy link
Member

@rullzer great stuff, thank you! :)

@PVince81
Copy link
Contributor Author

We might need an icon/avatar as well for server to server shares (aka ownCloud federation):
There's currently an empty space there as well. (AFAIK we currently cannot download avatars from a remote server)

@jancborchardt
Copy link
Member

Would be awesome if we can then. :) At least for people whom you already shared with in the past.

@PVince81
Copy link
Contributor Author

Probably not in this ticket. I raised this one for that: #14564

@jancborchardt
Copy link
Member

Great stuff, thanks!

@nickvergessen
Copy link
Contributor

Could put it directly infront of the user/group name, just like the activity app does?

bildschirmfoto vom 2015-03-02 17 15 00

@rullzer rullzer closed this as completed Mar 16, 2015
@MorrisJobke MorrisJobke added this to the 8.1-current milestone Mar 16, 2015
@lock lock bot locked as resolved and limited conversation to collaborators Aug 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants