Skip to content

fix(feeds): restore feed management table layout - #1040

Merged
WCY-dt merged 1 commit into
DevXDojo:release/v1.3.28from
marcomarcogd:codex/fix-feed-management-layout
Aug 26, 2026
Merged

fix(feeds): restore feed management table layout#1040
WCY-dt merged 1 commit into
DevXDojo:release/v1.3.28from
marcomarcogd:codex/fix-feed-management-layout

Conversation

@marcomarcogd

Copy link
Copy Markdown
Contributor

Description

Restore a stable, aligned feed-management table in Settings and keep row interaction inside the settings workflow. This addresses the layout shown in #1023, where feed fields are scattered vertically and column values no longer line up.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • 🎨 Style/UI change
  • ✅ Test addition/update

Related Issues

Fixes #1023

Changes Made

  • Render checkbox, icon, name/source, category, latest article, frequency, status, and actions in one shared grid definition.
  • Preserve the API/user order by default and move optional sorting into a compact selector.
  • Keep all columns available at narrow widths by scrolling only the table region horizontally.
  • Truncate long names, URLs, and categories while retaining full values in native tooltips.
  • Use existing feed images first and provide a stable RSS fallback.
  • Open the feed editor from editable rows without closing Settings or changing the main-view feed selection.
  • Stop checkbox/edit/delete events from triggering the row action and keep FreshRSS-managed rows locked.
  • Preserve the last successful list on refresh errors and add loading, empty, error, and retry states.

Testing

Test Configuration

  • OS: macOS 15
  • MrRSS Version: current main
  • Node Version: 24 for lint/build

Test Steps

  1. Run cd frontend && npm run lint.
  2. Run cd frontend && npm run build.
  3. Run npx cypress run --spec cypress/e2e/feed-management.cy.ts.
  4. Verify long Chinese/English text, 120 feeds, errors, sorting, row edit behavior, and empty tag responses.

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective
  • New and existing targeted tests pass locally with my changes
  • This change has no dependent changes

Additional Notes

The target Cypress spec passes all 13 tests. Feed APIs, database schema, persisted ordering, and FreshRSS ownership behavior are unchanged.

Breaking Changes

None.

@WCY-dt
WCY-dt changed the base branch from main to release/v1.3.28 August 25, 2026 14:16
@marcomarcogd
marcomarcogd marked this pull request as ready for review August 26, 2026 00:52
@WCY-dt
WCY-dt merged commit 085e9ff into DevXDojo:release/v1.3.28 Aug 26, 2026
4 checks passed
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.

[STYLE] 设置订阅管理有问题

2 participants