FreshRSS icon indicating copy to clipboard operation
FreshRSS copied to clipboard

OPML category is a text field

Open Frenzie opened this issue 2 years ago • 8 comments

Describe the bug Category is an input

To Reproduce Steps to reproduce the behavior:

  1. Go to add subscription or category

Expected behavior It should be a drop down

Screenshots image

Environment information (please complete the following information):

  • FreshRSS version: 1.20.0

Frenzie avatar Nov 18 '22 12:11 Frenzie

That field is for the name of the new category associated to the dynamic OPML, but I can see that the naming of the field could/should be improved

Alkarex avatar Nov 18 '22 12:11 Alkarex

An empty category is needed, isn't it?

math-GH avatar Nov 18 '22 14:11 math-GH

An empty category is needed, isn't it?

Yes, a new category of that name will be created to host the dynamic OPML

Alkarex avatar Nov 20 '22 16:11 Alkarex

How to proceed here?

math-GH avatar Apr 12 '24 18:04 math-GH

Is a unique type of category required or merely an empty category?

Frenzie avatar Apr 12 '24 19:04 Frenzie

A non-existing category name is expected at the moment

Alkarex avatar Apr 12 '24 21:04 Alkarex

What is expected to change here @Frenzie ?

math-GH avatar Apr 13 '24 14:04 math-GH

Primarily the string since it's confusing. Perhaps that could be as simple as something like Category name for OPML or OPML category. Or maybe the string can stay, but it could have some help text. Just like how the URL says it's dynamic, the category help text could say it's uniquely for OPML.

Ideally you'd also want to reject existing category names before even submitting. At the simplest a dynamic negative lookahead pattern would probably do the trick, though you'd want to set a custom rejection message.

Frenzie avatar Apr 13 '24 14:04 Frenzie