Skip to content

Add Tag page.#1354

Closed
jacac wants to merge 3 commits intoBookStackApp:masterfrom
jacac:bookstack_738
Closed

Add Tag page.#1354
jacac wants to merge 3 commits intoBookStackApp:masterfrom
jacac:bookstack_738

Conversation

@jacac
Copy link

@jacac jacac commented Mar 27, 2019

Regarding #738.

jacac added 3 commits March 27, 2019 09:22
Add translation tags.
Fixed sorting on tag list.
@jacac jacac changed the title Bookstack 738 Add Tag page. Apr 10, 2019
@jacac
Copy link
Author

jacac commented Apr 10, 2019

@ssddanbrown Any comments if this tag page could be added? I've attached a screenshot of the tag page.
image

@ssddanbrown
Copy link
Member

Thanks @jacac,
Sorry I have not responded sooner, This comes at a bit of an awkward time with the redesign (#1153) in progress.

It looks good! I think we'll need to evaluate the best place for this to sit in BookStack along with how we ensure it adds value. Will also need some edits to the code along with merging into the redesign.

Probably won't be until after the redesign release that I can properly review this.

@ssddanbrown ssddanbrown mentioned this pull request Nov 15, 2019
@fklappan
Copy link

fklappan commented Aug 2, 2020

Sorry for thread necroing, but is this still relevant/doable? Some kind of "tag UI" would be great.

@ssddanbrown
Copy link
Member

Since this is relatively old I'm going to close this off, Although I appreciate that's due to my lack of review.

I'm still open to such a page being added, but it'd need to have a little more purpose than simply a list of tag names. I think the number of usages of that tag and the number of unique values used, for that tag, would be a good start since then it can provide an idea of utilisation.

In terms of UI, A paginated tabular list may work better here than being alphabetical breakdown, especially when considering potential emoji and non-ascii usage.

@jacac jacac deleted the bookstack_738 branch November 8, 2021 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants