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
  • 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 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.