Skip to content

feed update slider: <label> -> <div>#7882

Merged
Alkarex merged 1 commit intoFreshRSS:edgefrom
math-GH:fix/feed-update-label-tag
Aug 31, 2025
Merged

feed update slider: <label> -> <div>#7882
Alkarex merged 1 commit intoFreshRSS:edgefrom
math-GH:fix/feed-update-label-tag

Conversation

@math-GH
Copy link
Contributor

@math-GH math-GH commented Aug 30, 2025

Ref #7646

<label> needs to have an HTML input within or a for="" attribute

Changes proposed in this pull request:

  • fixed 2 <label> who are not a 'real' label, so a <div> is the better choice

How to test the feature manually:

  1. open feed configuration
  2. click on "Icon" (first lines of config page) or "Export as OPML" (last lines of config page)
  3. nothing will happen. Mouse cursor will not change while hovering over the labels.

@math-GH math-GH added this to the 1.28.0 milestone Aug 30, 2025
@math-GH math-GH added the UX User experience label Aug 30, 2025
@Inverle
Copy link
Member

Inverle commented Aug 30, 2025

Indeed

@Alkarex Alkarex merged commit 94d4f94 into FreshRSS:edge Aug 31, 2025
1 check passed
@math-GH math-GH deleted the fix/feed-update-label-tag branch August 31, 2025 21:16
@Alkarex Alkarex modified the milestones: 1.28.0, 1.27.1 Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UX User experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants