-
Notifications
You must be signed in to change notification settings - Fork 55
Avatar #51
Comments
@karlitschek yes, once you upload an avatar for your user (on "Personal" page) |
But we have a nice "default" avatar. A colored box with the first letter of the name in it. Shoudn't this be used instead of a smilie? |
We probably need to think about the possibility of using these profile pictures outside of the browser (like in the mobile app) – and even the placeholders. Maybe it’s good if placeholder.js also has a component to render these placeholders as png? @kabum albeit I would make the pure HTML+CSS stuff still default and the rendering only in addition for now. |
This will result in a full reimplementation of the placeholder code in php - meaning duplicate code and duplicate maintenance effort - generally speaking something we should try to avoid. @VicDeo no chance to inject the html/css stuff into webodf? THX |
It's cool to user html for that. Bu at the end of the day we need a real image for Desktop, Mobile and so on as Jan said. So we should provide a default image in the owncloud core. It's easy to do with php. @kabum |
Should it hooked into the preview stuff (because it's nothing other than that) or should it extend the avatar stuff. Or does they already have something common? |
@DeepDiver1975 there is nothing impossible at all. I avoid patching upstream as I don't want to create additional troubles with downstreaming in future. The other thing is that I foresee the pain of compiling WebODF on my own, but I think I can stand it. |
In WebODF we have plans how to improve things to allow users of webodf.js complete customization of the rendering, not just by data. But that is not on the current agenda, so nothing to rely on for OC5/OC6 releases. There are two places in webodf or rather webodf.js and the editor extensions where avatars are used: for the cursor flagging and in that member list view. About cursor flagging: About Member List View: About patches & stuff: |
@VicDeo okay, will do tonight |
Can we use the user avatar picture instead of the smilie default image?
I'm testing core master and documents master here.
@VicDeo
The text was updated successfully, but these errors were encountered: