Skip to content

Add the ability to collapse Environment section using a Hide/Unhide #622

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.

Sign up for GitHub

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

Jump to bottom
Closed
arindam31 opened this issue Apr 4, 2023 · 5 comments · Fixed by #638
Closed

Add the ability to collapse Environment section using a Hide/Unhide #622

arindam31 opened this issue Apr 4, 2023 · 5 comments · Fixed by #638

Comments

@arindam31
Copy link

In some projects, the environment variable list is just too long. Takes a bit of a scroll to reach the test details.

There can two solutions:

  1. Add a simple button

114417843-3f18b580-9bb2-11eb-8edb-5d338e484345-2

The JS will be also simple:

function hideEnvironment() {
  var x = document.getElementById("environment"); // 
  var buttonName = document.getElementById("environHideUnhide");
  if (x.style.display === "none") {
    x.style.display = "block";
    buttonName.innerText = 'Hide';
  } else {
    x.style.display = "none";
    buttonName.innerText = 'UnHide';
  }
}
  1. Add the ability to add JS and HTML tags anywhere with a hook. May be based on IDs.
@BeyondEvil
Copy link
Contributor

Maybe a query param to hide it by default would be good too?

@BeyondEvil
Copy link
Contributor

For 2, I've been pondering the idea of allowing users to pass in their own Jinja2 and app.js. Basically only using the plugin as a pure backend.

Need to think some more about this.

@BeyondEvil BeyondEvil mentioned this issue Apr 8, 2023
Merged
@BeyondEvil
Copy link
Contributor

Would you remind reviewing #638 @arindam31 ?

@wyardley
Copy link

wyardley commented Sep 6, 2023

Just curious - did there end up being a way to hide it by default via a config option, or just the mentioned options with custom templates?

@BeyondEvil
Copy link
Contributor

Just curious - did there end up being a way to hide it by default via a config option, or just the mentioned options with custom templates?

Currently, the only way of hiding it is clicking the ”button”.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feature: Add hide-able Environment Table BeyondEvil/pytest-html
3 participants
@arindam31 @wyardley @BeyondEvil