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

Cannot click 're-run all checks' on Actions page #2328

Closed
paambaati opened this issue Aug 14, 2019 · 4 comments · Fixed by #2433
Closed

Cannot click 're-run all checks' on Actions page #2328

paambaati opened this issue Aug 14, 2019 · 4 comments · Fixed by #2433
Labels
bug help wanted small Issues that new contributors can pick up

Comments

@paambaati
Copy link

If you're on the latest beta of GitHub Actions, there's a drop-down for re-running all checks that you can't click.

I can confirm that it works fine on Firefox and in Chrome after I've disabled the extension.

Example URL — https://github.com/paambaati/websight/pull/10/checks?check_run_id=193115347

Screenshot of the dropdown in question (from Firefox)

Screenshot 2019-08-14 at 2 41 10 PM

CSS selector

details.dropdown:nth-child(1) > summary:nth-child(1)

Markup

<div>
          <details class="dropdown details-reset details-overlay d-inline-block ml-4 mt-n1" open="">
            <summary class="btn btn-sm" aria-haspopup="true">
              <svg></svg>
              Re-run checks
              <div class="dropdown-caret"></div>
            </summary>

            <template id="js-check-suite-rerequest-form-success">
              <div class="flash flash-full flash-notice">
  <div class="container">
    <button class="flash-close js-flash-close" type="button" aria-label="Dismiss this message">
      <svg></svg>
    </button>
    
                You have successfully requested checks from GitHub Actions.

  </div>
</div>
            </template>

            <template id="js-check-suite-rerequest-form-error">
              <div class="flash flash-full flash-error">
  <div class="container">
    <button class="flash-close js-flash-close" type="button" aria-label="Dismiss this message">
      <svg></svg>
    </button>
    
                There was a problem requesting checks from GitHub Actions.

  </div>
</div>
            </template>

            <ul class="dropdown-menu dropdown-menu-sw" style="width: 170px">
              <li>
              </li>
              <li>
                    <input type="hidden" name="only_failed_check_runs" value="false">
                    <button class="dropdown-item btn-link" type="submit">
                      Re-run all checks
                    </button>
</form>              </li>
            </ul>
          </details>

    </div>
@paambaati paambaati added the bug label Aug 14, 2019
@fregante
Copy link
Member

What's its container? Is it inside .pr-toolbar?

@paambaati
Copy link
Author

@fregante It is, yes.

@notlmn
Copy link
Contributor

notlmn commented Aug 16, 2019

Had a chance to test this, it looks like close-out-of-view-modals is the culprit.

image

GitHub uses some <template> tags for internal UI use, that which https://github.com/sindresorhus/refined-github/blob/372e282cfd2e207704e99c4b4538e110b4a4d1a4/source/features/close-out-of-view-modals.tsx#L16 picksup, <template> tags are display: none by default, causing them to report as "out of view" in this case.

image


Out of context, GitHub should provide an option to re-run tests (individually if possible) for recent comments and PR that pass too, not just for PRs that have failed steps.

I don't who to ping about this, so pinging @lukehefson as a start.

@fregante
Copy link
Member

Not GitHub’s fault here, the feature should look for other siblings of summary, I suppose, that are not template

PR welcome

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug help wanted small Issues that new contributors can pick up
Development

Successfully merging a pull request may close this issue.

3 participants