brOOper / gwt-cal

Automatically exported from code.google.com/p/gwt-cal
0 stars 0 forks source link

Week view fails on IE when more then 14 events #78

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. It's from my own project but I'll try to reproduce on gwt-cal-demo
2. I was adding events (same day, same time, 2h duration) and after 14 events 
added, day view displaying 7 days fails on IE8 (it displays full-width 
appointments on FF).

What is the expected output? What do you see instead?
Events should be scaled or overlap somehow, but it fails. I'm getting 
javascript error in line:
      panel.com_google_gwt_user_client_ui_UIObject_element.style[$intern_116] = appt.com_bradrydzewski_gwt_calendar_client_dayview_AppointmentAdapter_width + $intern_152;

Unfortunately cannot run hosted IE on my project so it's detailed javascript 
output only.

What version of the product are you using? On what operating system?
0.9.1

Please provide any additional information below.
It looks like width getting too small and can't be displayed on IE.It's ignored 
on FF.

Original issue reported on code.google.com by jakub.gr...@gmail.com on 16 Aug 2010 at 12:27

GoogleCodeExporter commented 8 years ago
I hooked up AppointmentAdapter to check whether it's really too small width 
problem and it looks like it's confirmed.

Putting (quite-ugly):
if (width > 10) {
this.width = width;
} else {
width = 40;
}
piece of code in setWidth of AppointmentAdapter I was able to workaround this 
issue.

It looks like a good idea to employ some kind of minWidth parameter to calendar 
(probably strategy is good place to put it in).

Original comment by jakub.gr...@gmail.com on 16 Aug 2010 at 1:58

GoogleCodeExporter commented 8 years ago
Width is calculated as % value and in my case it was: -0.107142806.

Original comment by jakub.gr...@gmail.com on 16 Aug 2010 at 2:16

GoogleCodeExporter commented 8 years ago
Hey Jakub, how would you like to see this handled? Obviously the space is too 
small to show all 14 appointments ... so should we only show a subset? And show 
some type of indication that some are not currently visible?

How does outlook address this problem? How does google calendar address this 
problem?

Original comment by Brad.Ryd...@gmail.com on 18 Aug 2010 at 6:07

GoogleCodeExporter commented 8 years ago
Yes it's too small to show them separated with spacing like in current 
strategy. But google makes them overlap when there are many. 
I was playing with the code and I was able to make them overlap (no changes in 
positioning, just added miminum width). When you put higher z-index to selected 
appointments style it's also perfectly usable to navigate through them by 
arrows. Selected one shows on top. It's not eye-candy, but is usable. Please 
look at the file attached.

Original comment by jakub.gr...@gmail.com on 18 Aug 2010 at 7:47

Attachments: