Hi,

I have spent some time this evening getting Rounded Corners to work in Drupal 6, at the moment I don't have CVS access setup so I have included the changed files in the attached zip file.

Hope this is of help.

CommentFileSizeAuthor
#6 227260_roundedcorners_d6.patch1.97 KBchristefano
#6 rounded_corners.zip7.91 KBchristefano
#3 yh.patch1.85 KByhager
rounded_corners.zip1.83 KBRdN

Comments

zilla’s picture

zilla commented

this is great - what were the changes you made? anything substantial?

RdN’s picture

RdN commented

Hi zilla,

sorry for the delay in replying I have been busy with other stuff and this slipped out of view.

There weren't that many changes, there is not much Drupal code with it being mainly a wrapper. I will try and get a CVS update file created soon.

yhager’s picture

yhager commented
Status: Active » Reviewed & tested by the community
StatusFileSize
new yh.patch1.85 KB

Here is a patch against latest D5 CVS.
It produces the same code as written by @RdN, so I marked this as RTBC.

Anonymous’s picture

Anonymous (not verified) commented

Patch works for me.

Dnicolas’s picture

Dnicolas commented

rounded_corners.zip works perfect for me.

christefano’s picture

christefano commented
StatusFileSize
new rounded_corners.zip7.91 KB
new 227260_roundedcorners_d6.patch1.97 KB

#3 didn't apply successfully. Attached is a patch that updates the current 5.x-1.x-dev release to Drupal 6. I haven't tested it yet.

Also attached is a zip file containing the updated Drupal 6 version.

christefano’s picture

christefano commented
Status: Reviewed & tested by the community » Needs review

Needs review. This update was done at Drupal Café with Sim_M, design_dog and Benno Sebastian.

guillaumeduveau’s picture

guillaumeduveau
Location Toulouse
commented

Hi Christefano,

Your last .zip for D6 seems to work very fine for me !

Andrew Schulman’s picture

Andrew Schulman commented

Subscribing

christefano’s picture

christefano commented

Please mark this "reviewed & tested" if it's working for you. Thanks!

guillaumeduveau’s picture

guillaumeduveau
Location Toulouse
commented
Status: Needs review » Reviewed & tested by the community

Here it is :)

joachim’s picture

joachim commented

+1 works like a dream.
Please could someone commit?

pribeh’s picture

pribeh commented

It looks like this module is almost abandoned so sorry for hijacking this thread but does anyone who's worked on the 6.0 version wanna take a stab at, or provide me with a pointer as to how, making this module use browser detect or conditional behavior for use with IE only. Since firefox and webkit browsers all support CSS3's rounded corners property now it's pretty much only IE that we need a javascript solution for. Any pointers would be appreciated - as I'm not a module developer.

yhager’s picture

yhager commented

Committed.

yhager’s picture

yhager commented
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.