css ids for blocks [#73721]

The css id generated for the microcontent blocks are of the form "block-microcontent-93" (where 93 is the node id)

For "predefined" microcontents (mission, footer, etc...), having ids of the form "block-microcontent-mission" would provide a more robust way to theme them.

Comment	File	Size	Author
#3	microcontent.module.patch	1.85 KB	yched

Comments

Comment #1

yched commented 14 July 2006 at 15:09

edit : more precisely, since the block css id is generated by the theme layer form the block delta,
this would mean having an alphanumeric delta for these blocks.

Comment #2

Bèr Kessels commented 14 July 2006 at 16:19

Version:

4.7.x-1.x-dev

» master

Good idea. All it needs is a patch..... :)

Marked it for CVS, since I dont add features to a stable version (note that HEAD microcontent == 4.7 compatible)

Comment #3

yched commented 15 July 2006 at 21:17

Status:

Active

» Needs review

Status	File	Size
new	microcontent.module.patch	1.85 KB

Allright, here is the patch :-)

note that it will probably ruin the block settings the user had previously set...

Side note : Why do you use 'details' instead of 'footer' for the footer microcontent ?
I kept my patch coherent with that, but I found it strange...

Comment #4

Bèr Kessels commented 16 July 2006 at 10:34

Status:

Needs review

» Needs work

1. You are right about the previous settings. I fear we need to ship a .install containing a hook_update :(. setting to "needs work" because IMO it is critical not to break peoples code.

2. Footer indicates a placement, the bottom of the page. However, microcontent allows you to place your details anywere on the site. I chose the name after a naming convention from the world of CSS: http://www.stuffandnonsense.co.uk/archives/whats_in_a_name_pt2.html