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

Report UX and performance improvements #314

Merged
merged 40 commits into from
Oct 5, 2024

Conversation

jesseleite
Copy link
Member

@jesseleite jesseleite commented Jan 9, 2024

Users have mentioned that it's really hard to work through a report with thousands of pages, because actionable items get buried way down the report. Also if enough vue components are registered, the report view starts to chug.

This PR gives the whole SEO reporting section a facelift, puts more of a priority on showing actionable items first, and also solves the above-mentioned performance problems by paginating results.

BEFORE:

Old reports index...

CleanShot 2024-10-04 at 19 37 22

Old report details view...

CleanShot 2024-10-03 at 18 01 49

AFTER:

New reports index 💅

CleanShot 2024-10-04 at 19 37 55

New report details view 💅💅💅

CleanShot 2024-10-04 at 19 40 29


TODO:

  • Report generation
    • Cache actionable pages count at top level of generated report
      • Add test coverage
  • Reports listing
    • Add column headers
    • Add actionable pages column
    • Add pages crawled column
    • Fix status icon color to match site score status color on report show page
    • Fix dark mode
  • Report show page
    • Unify summary cards into one summary card
    • Make pages list a proper data-table
    • Add 'Actionable' column with pills showing what needs updating
      • Title
      • Description
      • Url
    • Allow sorting by column
      • Sort by status
        • Actionable items first by default
          • Failures, then warnings, then passing pages at bottom
      • Sort by url
    • Data table pagination for large data sets
    • Make url in actionable modal footer clickable
    • Add entry edit buttons to each row
    • Add entry edit buttons to actionable modals
    • Fix green colour styles
    • Improve warning colour styles
    • Fix dark mode

@jesseleite jesseleite marked this pull request as ready for review October 5, 2024 02:40
@jesseleite jesseleite merged commit 23afc52 into master Oct 5, 2024
24 checks passed
@jesseleite jesseleite deleted the feature/report-ux-improvements branch October 5, 2024 02:43
@jesseleite jesseleite changed the title Report UX improvements Report UX and performance improvements Oct 7, 2024
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