Ticket #4411 (closed defect: fixed)

Opened 4 years ago

Last modified 2 years ago

breadcrumbs.css contains invalid rule

Reported by: dirk.moebius@dorma.com Assigned to: hasienda
Priority: low Component: BreadCrumbsNavPlugin
Severity: minor Keywords: css typo
Cc: otaku42 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

Change History

11/27/09 03:12:06 changed by anonymous

Confirmed! Please update your code with this easy fix - it is invalidating other css which will cause others grief.

11/16/10 00:15:08 changed by hasienda

  • cc set to otaku42.
  • keywords set to css typo.

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.

(follow-up: ↓ 4 ) 11/07/11 22:38:58 changed by hasienda

  • owner changed from ixokai to hasienda.
  • status changed from new to assigned.

Thanks for reporting this.

I'll commit the fix to the repository, provided I'll get appropriate permission.

(in reply to: ↑ 3 ) 11/08/11 12:50:25 changed by otaku42

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.

11/08/11 19:03:13 changed by hasienda

  • status changed from assigned to 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.

11/08/11 19:31:23 changed by hasienda

(In [10886]) BreadCrumbsNavPlugin: Shorten plugin package path and other code cleanup, refs #4411.

11/09/11 00:12:40 changed by hasienda

(In [10888]) BreadCrumbsNavPlugin: Correct CSS definition, refs #4411.

Thanks to Dirk Moebius for the report.

11/09/11 21:01:59 changed by hasienda

  • status changed from new to closed.
  • resolution set to fixed.

(In [10896]) BreadCrumbsNavPlugin: Tag long overdue maintenance release 0.2, closes #2463, #3965, #4037, #4411, #7036 and #7231.

Fix erroneously introduced version mismatch between changelog and setup.py.


Add/Change #4411 (breadcrumbs.css contains invalid rule)




Change Properties
Action