Date with event isn't highlighted in the calendar

Bug #740980 reported by Matthew Paul Thomas on 2011-03-23
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Unity Foundations
Karl Lattimer
indicator-datetime (Ubuntu)
Karl Lattimer

Bug Description

Binary package hint: indicator-datetime

indicator-datetime 0.1.98-0ubuntu1, Ubuntu Natty

1. In "Time & Date Settings…" > "Clock", check both "Monthly calendar" and "Coming events" if they are not checked already.
2. In Evolution, add an event for some day in the current month.
3. If bug 740977 has not been fixed yet, log out and log back in again.
4. Open the clock menu.

What you see: No special highlighting for that date.

What you should see: "If a date has at least one event, it should be shown in bold." <>

Related branches

Karl Lattimer (karl-qdh) wrote :

This should be working already, but I noticed it isn't.

I've been debugging the issue along with;

Karl Lattimer (karl-qdh) on 2011-03-23
Changed in indicator-datetime (Ubuntu):
status: New → In Progress
assignee: nobody → Karl Lattimer (karl-qdh)
Karl Lattimer (karl-qdh) wrote :

I've created the following bug to group these calendar widget related issues;

"IDO calendar should replace GtkCalendar wrapper with a grid layout menu"

I'm trying to fix this issue independent of the above bug, there seems to be a communication problem with dbusmenu at the moment.

Karl Lattimer (karl-qdh) wrote :

Some notes on this bug;

In datetime-service.c the function update_appointment_menu_items will call in the iteration of appointment items, dbusmenu_menuitem_property_set_int (calendar, CALENDAR_MENUITEM_PROP_MARK, dmday);

This is supposed to be received by calendar_prop_change_cb in indicator-datetime.c we assign this callback in new_calendar_item like so;

g_signal_connect(G_OBJECT(newitem), DBUSMENU_MENUITEM_SIGNAL_PROPERTY_CHANGED, G_CALLBACK(calendar_prop_change_cb), ido);

There are various properties we want to be able to set on the calendar, setting the date, marking, unmarking and clearing marks, each has a function in ido calendar menu item.

Watching the communication between the service and the indicator shows that a property call is being sent for set mark and clear marks. Neither of these commands appears to be received by indicator-datetime.c

David Barth (dbarth) on 2011-03-28
Changed in unity-foundations:
milestone: none → unity-3.8.0-beta
assignee: nobody → Karl Lattimer (karl-qdh)
importance: Undecided → Medium
status: New → In Progress
Karl Lattimer (karl-qdh) wrote :

Ted gave me a pointer to a possible problem, in that it might be de-duplicating the signals, in which case it would be better to fix this by sending an array of ints.

However, this still doesn't explain why the signal wasn't being received at all by the indicator. The recommendation there is to turn on -DMASSIVEDEBUGGIN in dbusmenu and see if there's a reason for it.

Possibly the array of ints will solve the problem, I'm trying that as it's a sensible move anyway.

Karl Lattimer (karl-qdh) wrote :

I've updated the branch to include the array of ints change tedg suggested, however the bug is still there.

Can get a capture from dbus-monitor but it looks like I'll need to rebuild dbusmenu to do any real testing on it.

Karl Lattimer (karl-qdh) wrote :

after some debugging I think I've cracked it, stay tuned :)

Karl Lattimer (karl-qdh) wrote :

Currently pushed branch shows marked days when the calendar is updated however it seems to fail to do this on startup.

I'm testing to see if it updates on the timer if so i'll add a one time event on startup to trigger an update after 30 seconds or so.

maybe ted will have a better suggestion

Karl Lattimer (karl-qdh) wrote :

Seems fixed in head today, thanks ted :D

Changed in unity-foundations:
status: In Progress → Fix Committed
Changed in indicator-datetime (Ubuntu):
status: In Progress → Fix Committed
status: Fix Committed → In Progress
Karl Lattimer (karl-qdh) on 2011-04-01
Changed in unity-foundations:
status: Fix Committed → Fix Released
Changed in indicator-datetime (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers