~netlandish/links#47: 
Tag filtering improvements. Adding AND and EXCLUDE functionality

Just a follow up from our chat earlier.

Right now we have the search box at the top right. Maybe we add a smaller text link under it that says "Advanced Search".

When clicking advanced it shows a new row below the search box with 2 text inputs. One for "include tags" and one for "exclude tags", both are auto complete.

When clicking on a tag (ie, so it goes /?tag=X) the "include tag" text field automatically has the tag there included so if someone clicks advanced it's already populated.

When viewing a filtered tag page (ie, /?tag=X) then whenever printing the tag "X" (in html) don't make it clickable (we're already on that tag filtered page). If clicking on another tag (say, tag "Z") from an already filtered page, let's combine the tags (an AND situation) so when clicking on it, we're seeing tags "X" AND "Z".

Status
RESOLVED IMPLEMENTED
Submitter
~petersanchez
Assigned to
Submitted
9 months ago
Updated
9 months ago
Labels
No labels applied.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit a748803.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 6a3be70.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 3168558.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 7a10396.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 2b4b1ac.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 9d9316a.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 50e3aa2.

~yaderv 9 months ago*

Comments by Peter

  • Make the form be linear. So like the included tags are too the left of the section/table, and the excluded is to the right. They can be large enough to fill the entire table space.
  • When a tag is selected (ie, in the query string) the filter form should be shown on page load
  • Note the tag 3 exclusion isn't working. That's because in the query string it has exclude=tag 3 (note the actual tag should be tag-3 so they need to be run through slugify or something before being sent for filtering
  • The current tag(s) are still clickable.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit d0f1260.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit a050631.

~yaderv 9 months ago

More comments by Peter:

  • The "Include tags" input box should be ABOVE the words "All, Unread, etc." on the left. It can be like 40% of the entire white box area where the links are listed.
  • The Exlucde should be next to it, also say 40% width.
  • The buttons next to the input boxes (10% each), Apply first, then clear.
  • It should be all one line (linear)
  • It seems to call directly to /home on the form. Is that correct? What if a user is viewing the say /yader-links slug and wants to filter or is filter only for the owner?

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 63dc457.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 599a912.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 47f92b0.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit aa7aaa3.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 360d339.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 0b0bf23.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 6190947.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 45dc131.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 4db1863.

~yaderv 9 months ago

Yader Velasquez referenced this ticket in commit 8d2e62f.

~petersanchez REPORTED IMPLEMENTED 9 months ago

Register here or Log in to comment, or comment via email.