Custom format does not handle html formatting any more

Bug #1235990 reported by HorstBort
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
libindicator
Won't Fix
Wishlist
Unassigned
indicator-datetime (Ubuntu)
Won't Fix
Wishlist
Unassigned

Bug Description

Until Ubuntu 13.04, the format of date and time in indicator-datetime could be formatted with HTML tags, like explained e.g. here: http://www.omgubuntu.co.uk/2010/10/how-to-customize-the-clock-applet-in-ubuntu

Since 13.10, this behaviour is not available any more.

Downgrading indicator-datetime to 12.10.3daily13.03.26-0ubuntu1 brings back the functionality in Gnome Flashback, but not in Unity.

ProblemType: Bug
DistroRelease: Ubuntu 13.10
Package: indicator-datetime 12.10.3daily13.03.26-0ubuntu1
ProcVersionSignature: Ubuntu 3.11.0-11.17-generic 3.11.3
Uname: Linux 3.11.0-11-generic x86_64
ApportVersion: 2.12.5-0ubuntu1
Architecture: amd64
Date: Sun Oct 6 17:06:40 2013
InstallationDate: Installed on 2011-12-02 (673 days ago)
InstallationMedia: Ubuntu 11.10 "Oneiric Ocelot" - Release amd64 (20111012)
MarkForUpload: True
SourcePackage: indicator-datetime
UpgradeStatus: Upgraded to saucy on 2013-09-14 (21 days ago)

Revision history for this message
HorstBort (haukemuentinga) wrote :
Charles Kerr (charlesk)
Changed in indicator-datetime (Ubuntu):
status: New → Confirmed
Changed in libindicator:
status: New → Confirmed
importance: Undecided → Wishlist
Changed in indicator-datetime (Ubuntu):
importance: Undecided → Wishlist
Revision history for this message
Charles Kerr (charlesk) wrote :

This was never a supported feature, but it's a neat idea and I wouldn't be against re-enabling this.

The old indicator-datetime code set the label text this way:

> gboolean use_markup = FALSE;
> if (pango_parse_markup(timestr, -1, 0, NULL, NULL, NULL, NULL))
> use_markup = TRUE;
>
> if (use_markup)
> gtk_label_set_markup(label, timestr);
> else
> gtk_label_set_text(label, timestr);

While it's now done in libindicator's indicator-ng.c this way:

> static void
> indicator_ng_set_label (IndicatorNg *self,
> const gchar *label)
> {
> if (label == NULL || *label == '\0')
> {
> if (self->entry.label)
> gtk_widget_hide (GTK_WIDGET (self->entry.label));
> return;
> }
>
> gtk_label_set_label (GTK_LABEL (self->entry.label), label);
> gtk_widget_show (GTK_WIDGET (self->entry.label));
> }

We could re-add pango markup support by updating indicator_ng_set_label to use the _set_markup() function when appropriate.

Revision history for this message
Charles Kerr (charlesk) wrote :

Actually, it looks like this broke at some point in unity7 too.

Given that nobody else is asking for this, and since the work to support it is growing past the one-paragraph-of-code level of effort, I'm going to mark this as a WontFix.

Changed in libindicator:
status: Confirmed → Won't Fix
Changed in indicator-datetime (Ubuntu):
status: Confirmed → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.