Opened 16 years ago
Closed 13 years ago
#4411 closed defect (fixed)
breadcrumbs.css contains invalid rule
Reported by: | Owned by: | Steffen Hoffmann | |
---|---|---|---|
Priority: | low | Component: | BreadCrumbsNavPlugin |
Severity: | minor | Keywords: | css typo |
Cc: | Michael Renzmann | Trac Release: | 0.11 |
Description
The rule
#breadcrumbs li ul
is commented out with /* ... */, but the trailing '}' is not commented out, producing an invalid selector which is detected by FireBug.
Revision 3065.
Attachments (0)
Change History (8)
comment:1 Changed 15 years ago by
comment:2 Changed 14 years ago by
Cc: | Michael Renzmann added; anonymous removed |
---|---|
Keywords: | css typo added |
Confirm, just noticed that the same way and fixed it last too. But did not investigate further, so before reading this ticket I thought I had introduced this on my own before.
Anyone with access to the SVN here? Seem like this plugin is little short of support.
comment:3 follow-up: 4 Changed 13 years ago by
Owner: | changed from Stephen Hansen to Steffen Hoffmann |
---|---|
Status: | new → assigned |
Thanks for reporting this.
I'll commit the fix to the repository, provided I'll get appropriate permission.
comment:4 Changed 13 years ago by
Replying to hasienda:
provided I'll get appropriate permission.
It seems you have been set up as maintainer of this plugin, so you already have commit permissions.
comment:5 Changed 13 years ago by
Status: | assigned → new |
---|
Oh, really? Than I must have missed a notification about this. Thanks for telling me. I just hesitate to push changes out in hit or miss fashion.
So I'll go and finish the maintenance tackover procedure and then resolve tickets.
comment:6 Changed 13 years ago by
(In [10886]) BreadCrumbsNavPlugin: Shorten plugin package path and other code cleanup, refs #4411.
comment:7 Changed 13 years ago by
(In [10888]) BreadCrumbsNavPlugin: Correct CSS definition, refs #4411.
Thanks to Dirk Moebius for the report.
comment:8 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Confirmed! Please update your code with this easy fix - it is invalidating other css which will cause others grief.