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

Destroy button not fit on the page when Layout is Responsive #1699

Closed
edimossilva opened this issue Jul 6, 2020 · 2 comments · Fixed by #1701
Closed

Destroy button not fit on the page when Layout is Responsive #1699

edimossilva opened this issue Jul 6, 2020 · 2 comments · Fixed by #1701
Labels
feature new functionality that’s not yet implemented views-and-styles how administrate looks and is interacted with

Comments

@edimossilva
Copy link
Contributor

edimossilva commented Jul 6, 2020

  • Scenario

    • On show resource page, when the layout is responsive and the column labels have many letters
  • Issue

    • Destroy button not fit on the page.
  • Example

    • image
  • Proposal

    • Increase the min-width to .main-content
  • What versions are you running?

    • Rails - 6.0.0
    • administrate - 0.14.0
@edimossilva edimossilva added the bug breakages in functionality that is implemented label Jul 6, 2020
@nickcharlton
Copy link
Member

Administrate's layout doesn't try to be responsive to different browser widths and so far it's not been something that's been prioritised above other work.

…however it is something I'd like to get fixed, so thanks for opening this!

Do you think you'd be able to open a PR with your suggested fix?

@nickcharlton nickcharlton added feature new functionality that’s not yet implemented views-and-styles how administrate looks and is interacted with and removed bug breakages in functionality that is implemented labels Jul 6, 2020
@edimossilva
Copy link
Contributor Author

Hi @nickcharlton, yes, I will create a PR.

edimossilva added a commit to edimossilva/administrate that referenced this issue Jul 7, 2020
edimossilva added a commit to edimossilva/administrate that referenced this issue Jul 7, 2020
edimossilva added a commit to edimossilva/administrate that referenced this issue Jul 7, 2020
edimossilva added a commit to edimossilva/administrate that referenced this issue Jul 7, 2020
edimossilva added a commit to edimossilva/administrate that referenced this issue Jul 7, 2020
edimossilva added a commit to edimossilva/administrate that referenced this issue Jul 8, 2020
edimossilva added a commit to edimossilva/administrate that referenced this issue Jul 8, 2020
edimossilva added a commit to edimossilva/administrate that referenced this issue Aug 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature new functionality that’s not yet implemented views-and-styles how administrate looks and is interacted with
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants