Modify

Opened 6 years ago

Closed 3 years ago

#4411 closed defect (fixed)

breadcrumbs.css contains invalid rule

Reported by: dirk.moebius@… Owned by: 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 (0)

Change History (8)

comment:1 Changed 5 years ago by anonymous

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

comment:2 Changed 4 years ago by hasienda

  • Cc otaku42 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: Changed 3 years ago 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.

comment:4 in reply to: ↑ 3 Changed 3 years ago 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.

comment:5 Changed 3 years ago 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.

comment:6 Changed 3 years ago by hasienda

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

comment:7 Changed 3 years ago by hasienda

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

Thanks to Dirk Moebius for the report.

comment:8 Changed 3 years ago by hasienda

  • Resolution set to fixed
  • Status changed from new to closed

(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 Comment

Modify Ticket

Action
as closed The owner will remain hasienda.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.