Fix execute Query 'Results' and 'Query status' section controls gets truncated at 200% resize mode of 'Query1' blade (#1105)

* Fix a11y querytab results section zoom section issue

* Update test smapshot

* Update test snapshot

* Resolved test case issue
This commit is contained in:
vaidankarswapnil 2021-09-27 23:33:01 +05:30 committed by GitHub
parent f7e7240010
commit 6ca8e3c6f4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
7 changed files with 560 additions and 553 deletions

View File

@ -2357,6 +2357,8 @@ a:link {
height: 100%; height: 100%;
flex-grow: 1; flex-grow: 1;
overflow: hidden; overflow: hidden;
min-height: 300px;
overflow-y: scroll;
} }
.tabs { .tabs {
@ -2832,7 +2834,7 @@ a:link {
#explorerNotificationConsole { #explorerNotificationConsole {
z-index: 1000; z-index: 1000;
overflow-y:auto; overflow-y: auto;
overflow-x: clip; overflow-x: clip;
} }

View File

@ -83,11 +83,11 @@ exports[`ThroughputInputAutoPilotV3Component autopilot input visible 1`] = `
> >
The starting autoscale max RU/s will be determined by the system, based on the current manual throughput settings and storage of your resource. After autoscale has been enabled, you can change the max RU/s. The starting autoscale max RU/s will be determined by the system, based on the current manual throughput settings and storage of your resource. After autoscale has been enabled, you can change the max RU/s.
<a <StyledLinkBase
href="https://aka.ms/cosmos-autoscale-migration" href="https://aka.ms/cosmos-autoscale-migration"
> >
Learn more Learn more
</a> </StyledLinkBase>
</Text> </Text>
</StyledMessageBar> </StyledMessageBar>
<StyledChoiceGroup <StyledChoiceGroup

View File

@ -167,11 +167,11 @@ exports[`SettingsUtils functions render 1`] = `
> >
The starting autoscale max RU/s will be determined by the system, based on the current manual throughput settings and storage of your resource. After autoscale has been enabled, you can change the max RU/s. The starting autoscale max RU/s will be determined by the system, based on the current manual throughput settings and storage of your resource. After autoscale has been enabled, you can change the max RU/s.
<a <StyledLinkBase
href="https://aka.ms/cosmos-autoscale-migration" href="https://aka.ms/cosmos-autoscale-migration"
> >
Learn more Learn more
</a> </StyledLinkBase>
</Text> </Text>
<Text <Text
styles={ styles={

View File

@ -15,9 +15,9 @@
.graphExplorerContainer { .graphExplorerContainer {
margin-left: @SmallSpace; margin-left: @SmallSpace;
overflow-y:auto; overflow-y: auto;
overflow-x: clip; overflow-x: clip;
height:100%; height: 100%;
.flex-display(); .flex-display();
.flex-direction(); .flex-direction();
@ -61,7 +61,7 @@
.loadGraphHelper { .loadGraphHelper {
margin: auto; margin: auto;
text-align: center; text-align: center;
height:50%; height: 50%;
img { img {
width: @LoadGraphHelperWidth; width: @LoadGraphHelperWidth;
@ -73,7 +73,7 @@
margin: @DefaultSpace; margin: @DefaultSpace;
} }
.loadGraphBtn{ .loadGraphBtn {
margin-top: @LargeSpace; margin-top: @LargeSpace;
} }
} }
@ -95,17 +95,17 @@
} }
/* Override bootstrap's nav pills */ /* Override bootstrap's nav pills */
.nav-pills>li.active>a, .nav-pills > li.active > a,
.nav-pills>li.active>a:focus, .nav-pills > li.active > a:focus,
.nav-pills>li.active>a:hover { .nav-pills > li.active > a:hover {
background-color: @AccentMediumHigh; background-color: @AccentMediumHigh;
} }
.nav-pills>li>a { .nav-pills > li > a {
border-radius: 0px; border-radius: 0px;
} }
.nav>li>a { .nav > li > a {
padding: 5px 15px; padding: 5px 15px;
} }
@ -115,16 +115,16 @@
.typeahead__cancel-button { .typeahead__cancel-button {
top: 5px !important; top: 5px !important;
right: .4em !important; right: 0.4em !important;
} }
.queryMetricsSummary { .queryMetricsSummary {
margin: @LargeSpace @LargeSpace 0px @DefaultSpace; margin: @LargeSpace @LargeSpace 0px @DefaultSpace;
table-layout: fixed; table-layout: fixed;
display: block; display: block;
overflow-y: auto; overflow-y: auto;
overflow-x: hidden; overflow-x: hidden;
min-height: 100px;
.queryMetricsSummaryHead { .queryMetricsSummaryHead {
.flex-display(); .flex-display();
@ -145,7 +145,8 @@
font-size: 12px; font-size: 12px;
width: 100%; width: 100%;
.flex-display(); .flex-display();
th, td { th,
td {
padding: @DefaultSpace; padding: @DefaultSpace;
&:nth-child(1) { &:nth-child(1) {
@ -169,7 +170,6 @@
} }
} }
.graphContainer { .graphContainer {
overflow: hidden; overflow: hidden;
height: 100%; height: 100%;
@ -177,7 +177,6 @@
padding-top: (2 * @MediumSpace); padding-top: (2 * @MediumSpace);
.flex-display(); .flex-display();
.leftPane { .leftPane {
width: 200px; width: 200px;
padding: 0px 0px 0px @DefaultSpace; padding: 0px 0px 0px @DefaultSpace;
@ -225,8 +224,8 @@
.graphTitle { .graphTitle {
margin-top: @DefaultSpace; margin-top: @DefaultSpace;
.graphExpandCollapseBtn{ .graphExpandCollapseBtn {
padding:2px 6px 5px 6px; padding: 2px 6px 5px 6px;
margin-right: @SmallSpace; margin-right: @SmallSpace;
&:hover { &:hover {
@ -237,7 +236,7 @@
.active(); .active();
} }
img{ img {
.dataExplorerIcons(); .dataExplorerIcons();
} }
} }
@ -250,7 +249,7 @@
height: 100%; height: 100%;
.graphModal { .graphModal {
background-color: rgba(255, 255, 255, .7); background-color: rgba(255, 255, 255, 0.7);
position: absolute; position: absolute;
top: 0px; top: 0px;
left: 0px; left: 0px;
@ -283,7 +282,6 @@
min-height: 0px; min-height: 0px;
margin-top: 0px; margin-top: 0px;
.rightPaneHeader { .rightPaneHeader {
/* TODO: Hack to align the trashbox with the header for now. */ /* TODO: Hack to align the trashbox with the header for now. */
margin-top: -28px; margin-top: -28px;
@ -407,16 +405,16 @@
.propertyTableHeader { .propertyTableHeader {
font-weight: bold; font-weight: bold;
color: @DefaultFontColor ; color: @DefaultFontColor;
border-bottom: 1px solid @BaseMediumHigh; border-bottom: 1px solid @BaseMediumHigh;
} }
/* Override autocomplete stuff */ /* Override autocomplete stuff */
.typeahead__container { .typeahead__container {
font: @mediumFontSize 'Segoe UI' !important; font: @mediumFontSize "Segoe UI" !important;
input { input {
font: @mediumFontSize 'Segoe UI' !important; font: @mediumFontSize "Segoe UI" !important;
padding: 0px @SmallSpace !important; padding: 0px @SmallSpace !important;
} }
} }
@ -427,12 +425,12 @@
.typeahead__cancel-button { .typeahead__cancel-button {
top: 5px !important; top: 5px !important;
right: .4em !important; right: 0.4em !important;
} }
.rightPaneAddPropertyBtn { .rightPaneAddPropertyBtn {
padding: @DefaultSpace; padding: @DefaultSpace;
margin-left: -8px;/* TODO: Hack to align the addproperty button with the section content for now. */ margin-left: -8px; /* TODO: Hack to align the addproperty button with the section content for now. */
img { img {
margin: 0px @SmallSpace @SmallSpace 0px; margin: 0px @SmallSpace @SmallSpace 0px;
@ -451,6 +449,8 @@
.flex-direction(); .flex-direction();
overflow: hidden; overflow: hidden;
width: 100%; width: 100%;
min-height: 200px;
overflow: auto;
} }
.paneTitle { .paneTitle {
@ -587,7 +587,7 @@
.loadmore { .loadmore {
use.loadMoreIcon { use.loadMoreIcon {
fill:@GraphNodeDefaultColor; fill: @GraphNodeDefaultColor;
&.active { &.active {
fill: @AccentMediumHigh; fill: @AccentMediumHigh;
@ -609,8 +609,6 @@
opacity: 0; opacity: 0;
} }
} }
} }
/* scroll for leftpane, rightpane and newvertex pane*/ /* scroll for leftpane, rightpane and newvertex pane*/

View File

@ -29,6 +29,7 @@ describe("Table query select Panel", () => {
it("Should checked availableCheckbox by default", () => { it("Should checked availableCheckbox by default", () => {
const wrapper = mount(<TableQuerySelectPanel {...props} />); const wrapper = mount(<TableQuerySelectPanel {...props} />);
expect(wrapper.find("#availableCheckbox").first().props()).toEqual({ expect(wrapper.find("#availableCheckbox").first().props()).toEqual({
ariaPositionInSet: 0,
id: "availableCheckbox", id: "availableCheckbox",
label: "Available Columns", label: "Available Columns",
checked: true, checked: true,

View File

@ -37,12 +37,14 @@ exports[`Table query select Panel should render Default properly 1`] = `
className="column-select-view" className="column-select-view"
> >
<StyledCheckboxBase <StyledCheckboxBase
ariaPositionInSet={0}
checked={true} checked={true}
id="availableCheckbox" id="availableCheckbox"
label="Available Columns" label="Available Columns"
onChange={[Function]} onChange={[Function]}
> >
<CheckboxBase <CheckboxBase
ariaPositionInSet={0}
checked={true} checked={true}
id="availableCheckbox" id="availableCheckbox"
label="Available Columns" label="Available Columns"
@ -328,6 +330,7 @@ exports[`Table query select Panel should render Default properly 1`] = `
<input <input
aria-checked="true" aria-checked="true"
aria-label="Available Columns" aria-label="Available Columns"
aria-posinset={0}
checked={true} checked={true}
className="input-55" className="input-55"
data-ktp-execute-target={true} data-ktp-execute-target={true}
@ -646,6 +649,7 @@ exports[`Table query select Panel should render Default properly 1`] = `
</CheckboxBase> </CheckboxBase>
</StyledCheckboxBase> </StyledCheckboxBase>
<StyledCheckboxBase <StyledCheckboxBase
ariaPositionInSet={1}
checked={true} checked={true}
disabled={false} disabled={false}
key="" key=""
@ -653,6 +657,7 @@ exports[`Table query select Panel should render Default properly 1`] = `
onChange={[Function]} onChange={[Function]}
> >
<CheckboxBase <CheckboxBase
ariaPositionInSet={1}
checked={true} checked={true}
disabled={false} disabled={false}
label="" label=""
@ -939,6 +944,7 @@ exports[`Table query select Panel should render Default properly 1`] = `
aria-checked="true" aria-checked="true"
aria-disabled={false} aria-disabled={false}
aria-label="" aria-label=""
aria-posinset={1}
checked={true} checked={true}
className="input-55" className="input-55"
data-ktp-execute-target={true} data-ktp-execute-target={true}

View File

@ -999,7 +999,7 @@ export default class QueryTabComponent extends React.Component<IQueryTabComponen
"data-order": 2, "data-order": 2,
"data-title": "Query Stats", "data-title": "Query Stats",
}} }}
style={{ height: "100%" }} style={{ height: "100%", overflowY: "scroll" }}
> >
{this.state.allResultsMetadata.length > 0 && !this.state.error && ( {this.state.allResultsMetadata.length > 0 && !this.state.error && (
<div className="queryMetricsSummaryContainer"> <div className="queryMetricsSummaryContainer">