-
Notifications
You must be signed in to change notification settings - Fork 16
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
Show larger images on themes pages #55
Comments
Speaking from experience, working with images in Elgg is a bit of a torture. We will need an API for managing uploads, storing them in the filestore, rendering them client side. For every image or icon, we now have to take the following steps:
Not to mention all the trouble with hard-coded icon sizes, global sizes config that can not be filtered via hooks, no image manipulation library etc. Additionally, I do not think that picture-centric is necessary pretty. Content intensive sites might not care as much about images, and vice-versa having a 200x200 avatar in a summary view that has no description might look ugly. |
Man, agreed big time. Definitely need a better image solution. Find the issue on elgg/elgg and let's continue discussing there |
The purpose of themes, though, is the look. It should be really easy to tell if a theme has the look in going for. Best way to do that is with pictures. |
I was thinking something more like the image is the full width of the main Do you have thoughts about how this should look on mobile?
|
Yes, I imagine that on the new gallery tab. |
Want themes to be more pretty. More picture-centric in general
The text was updated successfully, but these errors were encountered: