I have a minor aesthetic suggestion for amadou. I am obviously not a design expert, so it's ok if you choose to ignore it.
It seems to me the way that the user pictures are placed above the title of a post in teaser list view creates a lot of unnecessary whitespace. Whitespace can be good, but it just looks awkward to me.
See our frontpage for example:
http://www.progressiveu.org
If the picture were on the left and the title, tags, submitted by, etc. were to the right of the picture, wouldn't the layout look better?
When you click on the title to go to node view, I can understand how the title should go at the top. Still, even in this view it seems like you could put the tags, submitted by, etc (is there a name for that collection of stuff?) to the right of the picture, again saving some space and looking a little tidier.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | avatars.png | 95.41 KB | jwolf |
Comments
Comment #1
jwolf commentedYou'll have to edit node.tpl.php - you'll have to move:
I strongly suggest that you search Drupal.org for the solution.
The following should point you in the right direction:
http://drupal.org/node/60404#comment-114859
http://drupal.org/node/99682
http://drupal.org/node/44699
Comment #2
Art Morgan commentedI can tell you are getting irritated by my suggestions, so I'll shut up now!
To be honest I don't have the time or inclination to become a theme developer, even though I have the utmost respect for those who do. I wish I had staff or a volunteer who did, but but it's not likely to happen for the next several months.
For awhile I was amazed at all the effort and attention you were giving to the theme and its users, so I thought maybe some feature requests might be useful to the community as a whole. I know in my own site the only way we truly get better is by listening to suggestions from users. I'm sure you are moving on to other things now, so can't blame you for leaving it as is.
Anyway, thanks for giving us a great theme!
Comment #3
jwolf commentedArt
I'm not getting irritated by your suggestions. I thoroughly enjoy being the Art Morgan / Progressive U lackey.
In fact I am considering renaming the Amadou theme to the Art-U theme.
As I have helped you with your site in the past, you know that I am more than happy to help anyone out who is having problems or issues with the theme. I also deeply appreciate those who help contribute to the theme with suggestions, comments and solutions. I have learned a great deal from those, like yourself, who have contributed to this theme.
However, I feel I have to draw a line when it comes to making specific modifications to the theme that benefits only one. The theme(s) are packaged "as-is" and any custom modifications are solely the responsibility of the users and site admins who wish to use the theme(s).
I'm not moving on to other things. I am still actively maintaining this theme and I look forward to receiving more of your generous suggestions.
Please forgive me if I can not or will not make modifications specifically for Progressive U because other projects demand my attention and time.
I still love you! Don't go changing!
Comment #4
Art Morgan commentedOk, glad to hear I don't need to change too much ;-) I know the feeling of getting more suggestions than are necessary or practical!
I do appreciate all the pointers, and all the ones that I don't have time to get to myself I am archiving for the day when I have a full-time lackey.
Comment #5
jwolf commentedComment #6
jwolf commentedAdd this to your style.css
Me thinks the avatars look better on the right. :)
Comment #7
Art Morgan commentedSweet!
I tried it, and I like them better on the right too. I took out the .comments part for now, because I didn't like what that did to my comments, and I couldn't figure out how it was connected to the avatar change.
One thing I noticed though: in my case the Tags gets pushed down below the bottom of the picture, rather then wrapping in below the dateline. In your PNG you don't have this problem. You can see what I mean here:
http://www.progressiveu.org/node?page=1
Comment #8
jwolf commentedOh yeah. Sorry - forgot to mention that you should also change the .taxonomy class (line 247 - style.css) to clear:left