Skip to content

Commit 98c8ea4

Browse files
committed
remove unnecessary zIndex from Modals/PopoverMenus
Previously zIndex values were necessary for instructure-ui components to render correctly. This problem was solved in a recent release of instructure-ui so this workaround is no longer needed. Change-Id: I1bf21b1dde8ffe93109b2296e5acf03cdc2e011e Reviewed-on: https://gerrit.instructure.com/107550 Tested-by: Jenkins Reviewed-by: Shahbaz Javeed <sjaveed@instructure.com> Reviewed-by: Jennifer Stern <jstern@instructure.com> Product-Review: Derek Bender <djbender@instructure.com> QA-Review: Derek Bender <djbender@instructure.com>
1 parent 99938e6 commit 98c8ea4

10 files changed

Lines changed: 1 addition & 10 deletions

File tree

app/jsx/add_people/components/add_people.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -278,7 +278,6 @@ import APIError from './api_error'
278278
shouldCloseOnOverlayClick={false}
279279
size="medium"
280280
tabindex="-1"
281-
zIndex="9999"
282281
>
283282
<ModalHeader>
284283
<Heading tabIndex="-1">{I18n.t('Add People')}</Heading>

app/jsx/courses/CourseHomeDialog.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,6 @@ class CourseHomeDialog extends React.Component {
111111
return (<Modal
112112
isOpen={this.props.open}
113113
transition="fade"
114-
zIndex={99999}
115114
label={I18n.t('Choose Course Home Page')}
116115
closeButtonLabel={I18n.t("Close")}
117116
onReady={this.onReady}

app/jsx/eportfolios/MoveToDialog.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ import Select from 'instructure-ui/lib/components/Select'
8484
onReady={this.handleReady}
8585
onRequestClose={this.handleRequestClose}
8686
onClose={this.handleClose}
87-
zIndex='9999'>
87+
>
8888
<ModalHeader>
8989
<Heading>{ this.props.header }</Heading>
9090
</ModalHeader>

app/jsx/gradezilla/default_gradebook/components/ActionMenu.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,6 @@ import 'compiled/jquery.rails_flash_notifications'
180180

181181
return (
182182
<PopoverMenu
183-
zIndex="9999"
184183
trigger={
185184
<Button variant="link">
186185
<Typography {...buttonTypographyProps}>

app/jsx/gradezilla/default_gradebook/components/AssignmentColumnHeader.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,6 @@ class AssignmentColumnHeader extends React.Component {
195195
contentRef={this.bindOptionsMenuContent}
196196
focusTriggerOnClose={false}
197197
trigger={this.renderTrigger()}
198-
zIndex="9999"
199198
>
200199
<MenuItemGroup label={I18n.t('Sort by')}>
201200
<MenuItem

app/jsx/gradezilla/default_gradebook/components/AssignmentGroupColumnHeader.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,6 @@ class AssignmentGroupColumnHeader extends React.Component {
8888
contentRef={this.bindOptionsMenuContent}
8989
focusTriggerOnClose={false}
9090
trigger={renderTrigger(this.props.assignmentGroup)}
91-
zIndex="9999"
9291
>
9392
<MenuItemGroup label={I18n.t('Sort by')}>
9493
<MenuItem

app/jsx/gradezilla/default_gradebook/components/StudentColumnHeader.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,6 @@ export default class StudentColumnHeader extends React.Component {
103103
<PopoverMenu
104104
contentRef={this.bindOptionsMenuContent}
105105
focusTriggerOnClose={false}
106-
zIndex="9999"
107106
trigger={
108107
<span className="Gradebook__ColumnHeaderAction">
109108
<Typography weight="bold" fontStyle="normal" size="large" color="brand">

app/jsx/gradezilla/default_gradebook/components/TotalGradeColumnHeader.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,6 @@ class TotalGradeColumnHeader extends React.Component {
6969
contentRef={this.bindOptionsMenuContent}
7070
focusTriggerOnClose={false}
7171
trigger={renderTrigger()}
72-
zIndex="9999"
7372
>
7473
<MenuItemGroup label={I18n.t('Sort by')}>
7574
<MenuItem

app/jsx/outcomes/AddToCourseModal.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ export default React.createClass({
2525
size="auto"
2626
label={I18n.t("Modal Dialog: Add to course")}
2727
closeButtonLabel={I18n.t("Close")}
28-
zIndex="9999"
2928
ref={this._saveModal}
3029
onEntering={this._fixFocus}
3130
onClose={this.props.onClose}

app/jsx/shared/MessageStudents.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,6 @@ import Alert from 'instructure-ui/lib/components/Alert'
230230
label={this.props.title}
231231
onRequestClose={this.props.onRequestClose}
232232
closeButtonLabel={I18n.t('Close')}
233-
zIndex="9999"
234233
size='medium'
235234
onExited={this.props.onExited}
236235
>

0 commit comments

Comments
 (0)