Skip to content

Commit 6cfd899

Browse files
committed
Fix layout of breakpoint method selector
Previously this had slipped into a weirdly offset second line, I think due to flexbox card changes in WebRTC work. Just needs a container to ensure it's properly inline. Played briefly with making it full width, but I think short like this looks a little better.
1 parent bc524e9 commit 6cfd899

File tree

1 file changed

+19
-17
lines changed

1 file changed

+19
-17
lines changed

src/components/view/http/http-breakpoint-request-card.tsx

Lines changed: 19 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ const MethodSelect = styled(Select)`
4343
display: inline-block;
4444
4545
width: auto;
46-
margin-left: 10px;
46+
margin-left: 8px;
4747
margin-bottom: 5px;
4848
`;
4949

@@ -73,22 +73,24 @@ export class HttpBreakpointRequestCard extends React.Component<RequestBreakpoint
7373
</CollapsibleCardHeading>
7474
</header>
7575

76-
<ContentLabel>Method:</ContentLabel>
77-
<MethodSelect value={method} onChange={this.onMethodChanged}>
78-
{ !validMethods.includes(method as MethodName) &&
79-
<option key={method} value={undefined}>
80-
{ method }
81-
</option>
82-
}
83-
{ validMethods.map((methodOption) =>
84-
<option
85-
key={methodOption}
86-
value={methodOption}
87-
>
88-
{ methodOption }
89-
</option>
90-
) }
91-
</MethodSelect>
76+
<div>
77+
<ContentLabel>Method:</ContentLabel>
78+
<MethodSelect value={method} onChange={this.onMethodChanged}>
79+
{ !validMethods.includes(method as MethodName) &&
80+
<option key={method} value={undefined}>
81+
{ method }
82+
</option>
83+
}
84+
{ validMethods.map((methodOption) =>
85+
<option
86+
key={methodOption}
87+
value={methodOption}
88+
>
89+
{ methodOption }
90+
</option>
91+
) }
92+
</MethodSelect>
93+
</div>
9294

9395
<ContentLabelBlock>URL</ContentLabelBlock>
9496
<UrlInput value={url} onChange={this.onUrlChanged} />

0 commit comments

Comments
 (0)