After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 702849 - Calendar does not open
Calendar does not open
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2013-06-22 07:34 UTC by drago01
Modified: 2013-06-24 19:14 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
dateMenu: Update visibility syncing for new dateMenu layout (2.15 KB, patch)
2013-06-24 19:09 UTC, Jasper St. Pierre (not reading bugmail) 		committed Details | Review

Description drago01 2013-06-22 07:34:31 UTC 
Seems like clicking on the clock does not open the calendar in 3.9.3.
Comment 1 drago01 2013-06-23 16:38:15 UTC 
Seems like mouse wheel over the volume icon does not work as well. 
Jasper does this has anything to do with your focus rework? I could not spot any other related changes (did try to debug it though).
Comment 2 drago01 2013-06-23 21:52:11 UTC 
(In reply to comment #1)
> Seems like mouse wheel over the volume icon does not work as well. 
> Jasper does this has anything to do with your focus rework? I could not spot
> any other related changes (did try to debug it though).

OK this one is a fallout from: 847cb5b972a04e7dc21c362a

(gnome-shell-real:3647): Gjs-WARNING **: JS ERROR: TypeError: this._output.scroll is not a function
VolumeMenu<.scroll@/home/linux/gnome/install/share/gnome-shell/js/ui/status/volume.js:259
Comment 3 drago01 2013-06-23 21:57:01 UTC 
(In reply to comment #2)
> (In reply to comment #1)
> > Seems like mouse wheel over the volume icon does not work as well. 
> > Jasper does this has anything to do with your focus rework? I could not spot
> > any other related changes (did try to debug it though).
> 
> OK this one is a fallout from: 847cb5b972a04e7dc21c362a
> 
> (gnome-shell-real:3647): Gjs-WARNING **: JS ERROR: TypeError:
> this._output.scroll is not a function
> VolumeMenu<.scroll@/home/linux/gnome/install/share/gnome-shell/js/ui/status/volume.js:259

<drago01> real error is:
<drago01> (gnome-shell-real:7406): Gjs-WARNING **: JS ERROR: TypeError: this._slider.scroll is not a function
<drago01> StreamSlider<.scroll@/home/linux/gnome/install/share/gnome-shell/js/ui/status/volume.js:91
Comment 4 drago01 2013-06-23 22:01:26 UTC 
(In reply to comment #3)
> (In reply to comment #2)
> > (In reply to comment #1)
> > > Seems like mouse wheel over the volume icon does not work as well. 
> > > Jasper does this has anything to do with your focus rework? I could not spot
> > > any other related changes (did try to debug it though).
> > 
> > OK this one is a fallout from: 847cb5b972a04e7dc21c362a
> > 
> > (gnome-shell-real:3647): Gjs-WARNING **: JS ERROR: TypeError:
> > this._output.scroll is not a function
> > VolumeMenu<.scroll@/home/linux/gnome/install/share/gnome-shell/js/ui/status/volume.js:259
> 
> <drago01> real error is:
> <drago01> (gnome-shell-real:7406): Gjs-WARNING **: JS ERROR: TypeError:
> this._slider.scroll is not a function
> <drago01>
> StreamSlider<.scroll@/home/linux/gnome/install/share/gnome-shell/js/ui/status/volume.js:91

OK this one is now fixed by 5c25497
Comment 5 Jasper St. Pierre (not reading bugmail) 2013-06-24 19:09:33 UTC 
Created attachment 247672 [details] [review]
dateMenu: Update visibility syncing for new dateMenu layout

The code here before was trying to play hierarchy tricks to
figure out how to show / hide the events list, which broke
when we rearranged how the date menu was laid out. Simplify
the code here to not be so tricky, and update the CSS to
match the new designs.
Comment 6 drago01 2013-06-24 19:12:56 UTC 
Review of attachment 247672 [details] [review]:

LG and works fine here.
Comment 7 Jasper St. Pierre (not reading bugmail) 2013-06-24 19:14:21 UTC 
Attachment 247672 [details] pushed as 929636e - dateMenu: Update visibility syncing for new dateMenu layout