Skip to content

ref(issueDetails); Remove bottom margin on ConfigureDistributedTracing panel#37957

Closed
vuluongj20 wants to merge 1 commit into
masterfrom
vl/distributed-tracing-margins
Closed

ref(issueDetails); Remove bottom margin on ConfigureDistributedTracing panel#37957
vuluongj20 wants to merge 1 commit into
masterfrom
vl/distributed-tracing-margins

Conversation

@vuluongj20

Copy link
Copy Markdown
Contributor

Before: there's too much space between the panel and the error alert
Screen Shot 2022-08-17 at 2 56 22 PM

After:
Screen Shot 2022-08-17 at 2 56 50 PM

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Aug 17, 2022
background: none;
padding: ${space(2)};
margin: ${space(2)} 0;
margin: ${space(2)} 0 0;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this just be margin-top then?

@vuluongj20 vuluongj20 Aug 17, 2022

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Panel has its own set of margins, so we need to reset it here.

background: none;
padding: ${space(2)};
margin: ${space(2)} 0;
margin: ${space(2)} 0 0;

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What this looks like without the alert:
Screen Shot 2022-08-17 at 2 59 45 PM

@github-actions

github-actions Bot commented Sep 8, 2022

Copy link
Copy Markdown
Contributor

This pull request has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@github-actions github-actions Bot closed this Sep 16, 2022
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 1, 2022
@asottile-sentry asottile-sentry deleted the vl/distributed-tracing-margins branch December 1, 2023 16:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants