Bug 1297449 - Event edit dialog is unusable with a long attendee list;r=paenglab;a=philipp RELEASE_BASE_20160919
authormakemyday@gmx-topmail.de
Fri, 16 Sep 2016 11:02:38 +0200
changeset 25214 8f54e69715f50f817ced01988d8b1b236956d093
parent 25213 f392635ba8883345195b4584724dab01c5e08d53
child 25215 c2e06c1c53f80f456403e5040b992c456bc9ea88
push idunknown
push userunknown
push dateunknown
reviewerspaenglab, philipp
bugs1297449
Bug 1297449 - Event edit dialog is unusable with a long attendee list;r=paenglab;a=philipp
calendar/base/content/dialogs/calendar-summary-dialog.xul
calendar/base/themes/common/calendar-attendees.css
--- a/calendar/base/content/dialogs/calendar-summary-dialog.xul
+++ b/calendar/base/content/dialogs/calendar-summary-dialog.xul
@@ -241,21 +241,21 @@
         <img class="itip-icon"/>
         <label class="item-attendees-cell-label" crop="end" flex="1"/>
       </box>
       <box hidden="true" flex="1"/>
     </hbox>
   </vbox>
 
   <!-- Attendees -->
-  <box id="item-attendees" orient="vertical" hidden="true">
+  <box id="item-attendees" orient="vertical" hidden="true" flex="1">
     <spacer class="default-spacer"/>
     <calendar-caption label="&read.only.attendees.label;"
                       control="item-attendees-box"/>
-    <vbox id="item-attendees-box" dialogtype="summary">
+    <vbox id="item-attendees-box" flex="1">
       <spacer class="default-spacer"/>
     </vbox>
   </box>
 
   <!-- Description -->
   <box id="item-description-box" hidden="true" orient="vertical" flex="1">
     <spacer class="default-spacer"/>
     <calendar-caption label="&read.only.description.label;"
--- a/calendar/base/themes/common/calendar-attendees.css
+++ b/calendar/base/themes/common/calendar-attendees.css
@@ -1,25 +1,41 @@
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this file,
  * You can obtain one at http://mozilla.org/MPL/2.0/. */
 
+/* Begin: Summary Dialog - this should be moved to
+                           calendar/base/themes/common/dialogs/calendar-event-dialog.css
+                           or a separate file for the summary dialog */
+#calendar-summary-dialog,
+#calendar-event-summary-dialog,
+#calendar-task-summary-dialog {
+    min-width: 35em;
+}
+
+#calendar-summary-dialog #item-description,
+#calendar-event-summary-dialog #item-description,
+#calendar-task-summary-dialog #item-description {
+    min-height: 54px;
+}
+/* End: Summary Dialog */
 
 /* this is for attendee and organizer decoration in summary and event dialog */
 
 #item-attendees-box {
     -moz-appearance: listbox;
     margin: 2px 4px 0;
     overflow-y: auto;
-    min-height: 23px; /*at least two rows - otherwise a scrollbar (if required) wouldn't appear*/
+    min-height: 54px; /*at least two rows - otherwise a scrollbar (if required) wouldn't appear*/
 }
 
-#calendar-summary-dialog #item-attendees-box {
-    max-height: 92px; /* displays up to four rows of attendees*/
-    margin: 0px 10px;
+#calendar-summary-dialog #item-attendees,
+#calendar-event-summary-dialog #item-attendees,
+#calendar-task-summary-dialog #item-attendees {
+    max-height: 135px; /* displays up to four rows of attendees*/
 }
 
 .item-attendees-cell {
     padding: 3px 0px;
 }
 
 #calendar-event-dialog .item-attendees-cell {
     -moz-user-focus: normal;