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

checks: Consolidate check views #9718

Merged
merged 1 commit into from
Aug 10, 2023
Merged

checks: Consolidate check views #9718

merged 1 commit into from
Aug 10, 2023

Conversation

nijel
Copy link
Member

@nijel nijel commented Aug 10, 2023

Proposed changes

  • utilize path lookups
  • use one generic view to cover them all
  • list with progress for all views
  • provide specific views for all scopes instead doing filtering by params

Checklist

  • Lint and unit tests pass locally with my changes.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added documentation to describe my feature.
  • I have squashed my commits into logic units.
  • I have described the changes in the commit messages.

Other information

- utilize path lookups
- use one generic view to cover them all
- list with progress for all views
- provide specific views for all scopes instead doing filtering by
  params
@nijel nijel added this to the 5.0 milestone Aug 10, 2023
@nijel nijel requested a review from orangesunny as a code owner August 10, 2023 14:48
Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

<tr data-parent="row-{{ row.pk }}">
<td colspan="5" class="progress-row autocolspan">
<div class="progress">
<div class="progress-bar progress-bar-danger" role="progressbar" aria-valuenow="{{ row.check_progress }}" aria-valuemin="0" aria-valuemax="100" style="width: {{ row.check_progress }}%;"></div>
Copy link

Choose a reason for hiding this comment

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

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.

@nijel nijel merged commit f071e6f into WeblateOrg:main Aug 10, 2023
@nijel nijel deleted the check-views branch August 10, 2023 17:41
@nijel nijel mentioned this pull request Aug 11, 2023
5 tasks
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.

1 participant