Closed (won't fix)
Project:
BlueBreeze
Version:
5.x-1.2
Component:
CSS
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Feb 2007 at 00:34 UTC
Updated:
29 Aug 2013 at 17:09 UTC
Jump to comment: Most recent file
Comments
Comment #1
julianna commentedI have attached what it looks like in Firefox 1.5
Comment #2
julianna commentedI was able to fix the problem by replacing this:
with this (pulled from the Garland theme):
This code should be included in the next update for the theme.
Thanks.
Comment #3
Tobias Maier commentedHello julianna,
welcome at drupal and thank you for your first patch.
the first status is
patch (code needs review)and then someone hast to review the code.
as soon as other people think that the code is good to go the issue gets the status
patch (ready to be commited)btw. please create a proper patch -->
patch (code needs work)Thank you
Tobias
Comment #4
julianna commentedHi, Tobias.
Thanks for letting me know the next step. Unfortunately, I read through much of the patch documentation and it is more cryptic than Greek to me. I think I'll need to do much reading on CVS in general before I create a patch.
Julianna
Comment #5
julianna commentedOkay, I think I finally figured out this patch thing.
I'm leaving the status at "code needs work". Although the patch fixes the spacing issue, it moves the Primary Links boxes down a bit. This isn't an issue for me because I don't use the Primary Links but, obviously, this won't work for everyone.
Comment #6
Steel Rat commentedWhy do a patch for a theme instead of just correcting the theme? I'm confused.
Comment #7
julianna commentedBecause I'm a newb and don't know what I'm doing?
Comment #8
itapplication commented