Opened 14 years ago
Closed 14 years ago
#6277 closed bug (fixed)
Deskbar doesn't update time settings
| Reported by: | darkwyrm | Owned by: | zooey |
|---|---|---|---|
| Priority: | normal | Milestone: | R1 |
| Component: | Applications/Deskbar | Version: | R1/Development |
| Keywords: | Cc: | mdisreali@… | |
| Blocked By: | Blocking: | ||
| Platform: | All |
Description
If you change the time via the Time prefs app, the Deskbar still shows the time using the old time zone.
Time zone change was from Greenwich to US/Eastern in hrev37415 GCC4 Hybrid.
Change History (8)
comment:1 by , 14 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:2 by , 14 years ago
comment:3 by , 14 years ago
Why was there a need to have ICU control timezones? I find it confusing that the Time Preflet no longer does the job for which it is designed. I'm sure that many users will also be confused by this.
comment:4 by , 14 years ago
| Cc: | added |
|---|
comment:5 by , 14 years ago
Well I am sure anything not working to expectations is a temporary work in progress problem.
comment:6 by , 14 years ago
| Owner: | changed from to |
|---|
The time preflet is working properly, but the posix layer below is failing to take the changes into account.
comment:7 by , 14 years ago
| Version: | R1/alpha2 → R1/Development |
|---|
comment:8 by , 14 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Note:
See TracTickets
for help on using tickets.
Using the Locale kit and ICU, I had to drop the call to localtime(). This means ICU will now handle the timezone stuff. Unfortunately, it uses C++ objects, and these will not automagically update when you change some setting. So we'll have to tell ICU when a change of timezone occurs, so it can update the default BCountry object.
The other problem is the time preflet isn't aware of ICU yet, and could use it a lot more instead of messing with the timezone data directly. So it is not easy to plug them together.