Page <title> when reporting a bug is geeky and repetitive

Yes, this is terrible. My assumption is that this is a result of the changes from barry's heading rules change branch.

I presume then that the bug is not actually in malone?

After looking at the wiki rules closely for headings and titles, it seems we need to do a page_title on the view, which isn't yet being done, and specify a property on the view to override the reverse breadcrumbs. That makes this a bug in our malone view class, but it may be happening elsewhere. However, the wiki (https://dev.launchpad.net/VersionThreeDotO/UI/Conversion#Heading%20rules) says:

"Generally, you will not want to do this, so that the default reverse-breadcrumbs are used."

So I guess this title form should be the norm. I find it a little odd, though, at least on the filebug page. And I can't really imagine why we would want this for a page title anywhere. Having said that, I haven't noticed this anywhere but the +filebug page. So maybe the rules aren't exactly correct on the wiki. Maybe just having a page_title method on the view is enough.

On Wed, Sep 16, 2009 at 11:12 AM, Deryck Hodge
<email address hidden> wrote:
> After looking at the wiki rules closely for headings and titles, it
> seems we need to do a page_title on the view, which isn't yet being
> done, and specify a property on the view to override the reverse
> breadcrumbs.  That makes this a bug in our malone view class, but it may
> be happening elsewhere.  However,  the wiki
> (https://dev.launchpad.net/VersionThreeDotO/UI/Conversion#Heading%20rules)
> says:
>
> "Generally, you will not want to do this, so that the default reverse-
> breadcrumbs are used."
>
> So I guess this title form should be the norm.  I find it a little odd,
> though, at least on the filebug page.  And I can't really imagine why we
> would want this for a page title anywhere.  Having said that, I haven't
> noticed this anywhere but the +filebug page.  So maybe the rules aren't
> exactly correct on the wiki.  Maybe just having a page_title method on
> the view is enough.
>

I might have noticed other places, but I haven't been keeping a
record, nor filing bugs.

Perhaps it's worth asking on the mailing list whether this is the
expected behaviour?

jml

Martin has said that the <title> should be the reversed breadcrumbs, and that's what you'll get if you do not specify view.override_title_breadcrumbs=True. IIUC, Salgado has/is about to land a branch where the final +foo component is taken from the view.page_title if available. Note that you must set both attributes to override the reverse-breadcrumb in <title>.

I've updated the wiki rules to be (hopefully) clearer on this. I think it's still TBD whether setting view.page_title on every view can be relaxed with a better default. But I still think that setting override_title_breadcrumbs should be very rare. The base-layout rules should by default set the <title> correctly.

I think this is really an issue for Martin to settle.

Once my fix for bug 423691 lands, the title of that page will be "Reporting a bug in “indicator-session” package in Ubuntu : Bugs in indicator-session : “indicator-session” package : Ubuntu", which is certainly better than what we currently have (the one starting with +filebug).

I chatted with Martin and Barry about this and will respond on the dev list with the end result of that conversation. For the purpose of this bug, we'll use override_title_breadcrumbs to get a better human-readable title.

So that work will fall to we bugs hackers. I'll un-assign Martin.

While I don't think we fixed this as fully as mpt would like, i.e. returning to a simple string, there was some work to make this better. So the really bad "geeky and repetitive" part of this is fixed now. Maybe slightly repetitive still, but getting there. :)

Closing this as fixed released, since any further work is in the hands of registry as they and beuno decide whether or not to refine the rules for headings and page titles.

It is indeed less geeky, but it's now even *more* repetitive than it was before: "Report a bug about “indicator-session” package in Ubuntu : Bugs : “indicator-session” package : Ubuntu". Criminy.