Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use table-responsive for sample glob descriptions. #8974

Merged
merged 1 commit into from
Feb 1, 2022
Merged

Conversation

ryuyu
Copy link
Contributor

@ryuyu ryuyu commented Feb 1, 2022

Addresses #8919
This was being caused by a table in the API keys page being wider than the page when zoomed in.
This behaviour could also be replicated by making the window very narrow.
Update this table to use BootStrap's table-responsive, which causes the table content to correctly collapse with page width.

@ryuyu ryuyu requested a review from a team as a code owner February 1, 2022 00:18
@@ -457,7 +457,7 @@
<div class="panel-body">
<p>A glob pattern allows you to replace any sequence of characters with '*'.</p>
<p>Example glob patterns:</p>
<table class="table table-condensed borderless">
<table class="table-responsive table-condensed borderless">
Copy link
Member

Choose a reason for hiding this comment

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

In the docs it mentions giving the parent element .table-response. Does this work just the same?
https://getbootstrap.com/docs/3.3/css/#tables-responsive

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's a great question. I'll try this out too.

Copy link
Member

Choose a reason for hiding this comment

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

Spoke offline. LGTM.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Spoke with @joelverhagen "offline". It turns out that on the parent element, we have things that are not the table, so the behaviour is a little odd (scrollbar that can't be scrolled?). The current location has the behaviour we want to see (table resizes correctly to narrow window).

@ryuyu ryuyu merged commit f0697fd into dev Feb 1, 2022
@drewgillies drewgillies mentioned this pull request Feb 10, 2022
12 tasks
@ryuyu ryuyu deleted the ryuyu-fixApiKeySB branch March 16, 2024 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants