Just some little css enhancement [#336851]

Not sure if anyone else needs this, but since we deploy our own them for our site I needed some more divs with classes for the biblio result list and wrote the little patch.
a) It adds <div class="biblio-results"> around the whole list of results (no matter if they are inline, on profile or on the main page). This lets you style the background for biblio results or adjust all links inlcuding the sort and export links etc.

b) We wanted to hide the CSV export while marking the BibTex export link in red color. So I just put an own div around every export link, e.g.

'<span class="biblio-export-tagged">['.l("Tagged","$base/export/tagged"). ']</span>'.

This allows to hide some links using "display: none" or format themin any other way.

If anyone (including Ron, of course ;-)) considers this useful, here you are :-)

Comment	File	Size	Author
	biblio_css_enhancements.patch	1.87 KB	Frank Steiner

Comments

Comment #1

rjerome commented 20 November 2008 at 13:38

I'm thinking that perhaps rather than using CSS to do this, an admin setting could be added to fine tune which links are shown.

Comments...

Ron.

Comment #2

catdevrandom commented 20 November 2008 at 14:02

Why not both? :)

If the idea is just to select which items would be shown or not, then the admin setting is good enough, but to make some visually distinct presentation of the links (maybe icons or colors?), the div around the results would be a good approach, and would allow easier theming.

Maira

Comment #3

rjerome commented 20 November 2008 at 14:27

That's a good point, and it reminds me that once upon a time I was planning to wrap many more of the elements to allow for greater themeablilty (if that's a word).

Comment #4

Frank Steiner commented 20 November 2008 at 17:05

I didn't provide a patch for an admin option because I needed the color styling in addition, so I thought I would do everything with css.
But of course for people not using their own theme, an admin option would be great. I'm off now, but I can provide a combined patch for an option and css tomorrow.

However, I guess the same (css and config settings) should then be applied to the other views, too, like tabular, long view, short view? If want's to disable e.g. Tagged export I guess he/she would like to disable it everywhere.
What do you think?