Skip to content
This repository has been archived by the owner on Jul 2, 2024. It is now read-only.

Timeline in Invite Attendees is collapsed #14

Open
advancingu opened this issue Jun 7, 2017 · 9 comments
Open

Timeline in Invite Attendees is collapsed #14

advancingu opened this issue Jun 7, 2017 · 9 comments

Comments

@advancingu
Copy link
Member

In the Invite Attendees dialog, the timeline widget does not expand, making it impossible to read:
screenshot from 2017-06-07 21-12-18

@Trim
Copy link
Member

Trim commented Jun 8, 2017

That's quite strange. Could you please share with me the releases of:

  • Operating System (seems to be Ubuntu, but which one, 17.04?)
  • Thunderbird version: release 52.1.1, beta 54.0.2 ?
  • Lightning extension
  • exchangecalendar version and from where

That will help me to reproduce bug in my environment.

@advancingu
Copy link
Member Author

  • Thunderbird 52.1.1 64bit
  • Ubuntu 17.04 vanilla, default Unity desktop
  • Lightning 5.4
  • exchangecalendar v4.0.0-beta1 release file available here

Thanks for looking into it!

@advancingu
Copy link
Member Author

@Trim Ok, so I've debugged this a bit and found that it is caused by the flex="1" attribute on the first xul:listbox node under the freebusy-timebar. Once I delete the flex attribute and add CSS such as width: 100% on the listbox, the box expands as expected.

@Trim
Copy link
Member

Trim commented Jun 20, 2017

Oh sorry, I forgot to comment here. I have tested on two different computer (one physical and one virutal machine) with fresh Ubuntu 17.04 installation with exchangecalendar 4.0beta2 and I didn't see this issue.

Maybe the issue come from the Ubuntu theme after upgrade ?

According to the flex XUL attribute, it should already take all available space:

Flexible elements grow and shrink to fit their given space. Elements with larger flex values will be made larger than elements with lower flex values, at the ratio determined by the two elements.

To inspect the state of other elements in the dialog, you can use the DOM Inspector addon which even allows you to modify in live these attributes.

@advancingu
Copy link
Member Author

Yes, I debugged this in the DOM Inspector. As far as I could tell, the element only expanded when removing the flex attribute and adding width via CSS.
Also, can you tell if this is even an element from exchangecalendar? I couldn't find its definition in the code here, so it seems that this is from Lightning and that they need to fix this bug.

@Trim
Copy link
Member

Trim commented Jun 23, 2017

I've checked the code and we have an overlay on the calendar-event-attendees dialog.

However, this overlay only add some javascript code and a XUL binding.

Finally, commit 0d1a78a has removed the javascript code and now the overlay is still here but not used.

I'll do a final clean to remove correctly the overlay as it seems to not be used any more.

Edit: too speed written:
The overlay is used to add XBL code which adds a binding to the attendee list.
So I'll just remove the not available any more javascript load.

@advancingu
Copy link
Member Author

Not sure if I understand your comment. Does this mean you know how to fix the issue?

@Trim
Copy link
Member

Trim commented Jun 26, 2017

I wanted to say that freebusy-timebar is not modified by our code.

Although our code try to load some javascript file which were removed by the above mentioned commit.

We should clean up that, but I think that's not related to this issue.

@advancingu
Copy link
Member Author

So this element is from Lightning?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants