Skip to content

Inline commenting #1265

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
abbeyjackson opened this issue Feb 7, 2019 · 30 comments
Closed

Inline commenting #1265

abbeyjackson opened this issue Feb 7, 2019 · 30 comments

Comments

@abbeyjackson
Copy link

This is a suggestion added to issues as per Dan’s instructions via Twitter.

Inline commenting (selecting a chubk of text to comment on or having a comment associated with a specific line number) is a feature that is missing from Bookstack which I feel would really add value to the product.

@JohnnyWild89
Copy link

Looking forward to get this feature! Bookshelf is perfect replacement for any products which use as WIKI. Inline commentaries would greatly expands work possibilities.

@cze-karel
Copy link

This would be a really great feature that would improve the possibility of people collaborating on the documents. Is it possible this feature will make it into the roadmap?

@ssddanbrown
Copy link
Member

Further request and ideas proposed within #3063

@TiemeWoldman
Copy link

Any plans on adding annotations (Inline comments) to BookStack? This would be a great addition.

I tried hypothes.is for adding annotations to BookStack pages, but the BookStack text select pop-up interferes with hypothes.

@EDMJohnny
Copy link

As far as I can tell BookStack is pretty close to perfect for a writing / collaboration platform. Super simple to use, very intuitive. Inline comments would make it perfect! Is this something you are working on or have an ETA for?
Thanks

@ssddanbrown
Copy link
Member

@EDMJohnny

Is this something you are working on or have an ETA for?

Not at this time.

@EDMJohnny
Copy link

Thanks for responding. Is this something you'd be willing to put a bounty on / get paid for completing? Do you have a ballpark for cost of adding this feature?

@ssddanbrown
Copy link
Member

@EDMJohnny I'm not really a fan of public bounties, can just cause issues. Not really sure on ballpark costs either as this would need fleshing out into a full proposal since this request could be imagined/desired in different forms.

I've quoted features before (although never accepted) but only for features with clear scope and platform fit. Something makes me uneasy about rushing through a proposal for the sake of money, feel that risks providing an unbalanced larger voice to those with money, not sure the right way to balance that. If I was to quote it'd be somewhere upwards of £1k, even for a minimal implementation.

@danhansen
Copy link

Why not integrate something like https://github.com/recogito/recogito-js ? It looks like a significant portion of the functionality is already implemented.

@BloodyIron
Copy link

@ssddanbrown can we please have this added to a roadmap in some regard? As you may remember seeing me elsewhere, likely to switch to Bookstack, and some of our staff use this kind of feature in Confuence (which we are moving away from).

Is there a hurdle on your end for this?

@ssddanbrown
Copy link
Member

Is there a hurdle on your end for this?

Uh, I guess it's some level of formula of:

PRESSURE = ("user actual benefit" + "user demand" + "my own enthusiasm level for the feature")
DRAG = ("implementation cost" + "my predicated cost of maintenance, inc. support")
HURDLE = ((PRESSURE - DRAG) * "my current emotions" * RANDOM_CHANCE())

That's probably the best I can explain it, same for any feature tbh.
Roadmap is reserved for high-level elements. Could maybe have some kind of "Commenting and Communications" item. Not sure though, might want to address other things before.

Realistically, the next step for this specific request would be for me to put forward an implementation proposal, with a scope that I'd be happy to maintain, for community feedback. No reason to rush though IMO.

@ironbelly
Copy link

Agreed with EDMJohnny above that inline comments to facilitate more specific conversation amongst collaborators is one of the biggest items of value that could be added.

Also inline with EDMJohnny about the bounty, would be happy to chip in

@EDMJohnny
Copy link

@ssddanbrown Hi - just checking in again. I see you've done a lot of updates on Bookstack since I last checked it. All much appreciated. Any chance of bumping this feature request for inline comments on to your To Do?

@ipimpat
Copy link

ipimpat commented Mar 26, 2023

This would be a killer feature !

@rosequaartz
Copy link

This feature would be great! :)

@shashankvivek
Copy link

Yeah, with this feature, bookstack can knock the ball out of the park !

@stevenvegt
Copy link

I stumbled on this issue and wanted to chip in. We use bookstack for our open source community. People are working together on projects and keep reverting to google docs. Inline comments are the missing key in bookstack and holding them back to make the switch. I hope this will add some weight to the "user actual benefit" + "user demand" parts of the equation. Either way, thanks for this nice piece of software!

@dtufood-kihen
Copy link

Just dropping by to say I've been keeping an eye on the chat about inline comments. I get that developers have to juggle a bunch of stuff and can't just jump on every new idea, but I can't help feeling pretty stoked about this particular feature.

Please make it happen ! :-)

@khudari-starpath
Copy link

+1 would switch to bookstack if it had this feature!

@agapoar
Copy link

agapoar commented Jun 20, 2024

Hi. I would also love to see this feature added. By way of suggestion, I expected to see the ability to add an inline comment when I double clicked on a word in the text in read mode. I did see a popup menu, which was great. But obviously it didn't contain the ability to annotate. This screenshot shows the menu.

image

@ipimpat
Copy link

ipimpat commented Jun 20, 2024

Hi @ssddanbrown,

First, I want to express my gratitude for the incredible work.

I was wondering if you might consider adding the annotation feature to the roadmap. The demand for this feature seems quite high among users.

Currently, the absence of this feature is the main reason my team continues to rely on Word files. They are hesitant to transition to tools that lack annotation capabilities.

Thank you so much for considering this request, and for all the amazing work you continue to do.

@bendem
Copy link

bendem commented Jun 25, 2024

The demand for this feature seems quite high among users.

If you look at the most requested feature, it has >100 👍🏻. This one has 25.

Reminder that adding a 👍🏻 on the top post is the way to vote for an issue, pressuring/guilting the developer is not the way to get the feature you want added in.

image

@grv688
Copy link

grv688 commented Aug 1, 2024

Also very interested in this. We're migrating from documents that allowed inline commenting during revision cycles, and it was very useful.

@pato-lama
Copy link

Tossing a comment in here to support this request - would love having this functionality!

@ahnchive
Copy link

Also very interested in this!

@Yarrax
Copy link

Yarrax commented Dec 25, 2024

Also very interested in this!

@Yarrax
Copy link

Yarrax commented Feb 18, 2025

Hmmm... I'm sorry, but this feature is looks like most wanted:

Image

Of course, it haven't >100 thumbs up, but... :)

@sjimellor-jr
Copy link

Is Hypothes.is (https://web.hypothes.is/) an option? It can already be used to privately annotate Bookstack pages with an account and browser extension. Perhaps with a self-hosted "h" server (https://github.com/hypothesis/h) and using the JS client (https://github.com/hypothesis/client) configured to use the self-hosted server then this could be more native. I've not tried to host my own "h" server.

@Yarrax
Copy link

Yarrax commented Feb 19, 2025

Is Hypothes.is (https://web.hypothes.is/) an option?

Hmm... may be, thanks i'll try it. but anyway it will be a one more service for manual install. It will be great if bookstack will have inline commenting onboard

@ssddanbrown ssddanbrown mentioned this issue Apr 18, 2025
11 tasks
@ssddanbrown ssddanbrown added this to the Next Feature Release milestone May 22, 2025
@ssddanbrown
Copy link
Member

I've now added an initial implementation for in-content comments via PR #5584.
This is due to be part of the next feature release.
Therefore I'm going to close this off. Further related requests, or issues upon the implementation, can be raised in future requests if needed (once there's been opportunity to test after the next feature release).

This implementation is (relatively) minimal so may have some limitations in the view of some, but otherwise it should achieve the general functional goals for most. This implementation includes:

  • Ability to start a content-connected comment thread from the content popover when viewing a page.
  • The comment references a specific block of content, along with an optional text range within that to highlight a range of text.
  • Comment threads will show as highlighted ranges when viewing a page (as long as the referenced sections exist).
    • You can click the comment icon within a highlight range to open up the comment thread inline.
    • There's a button to jump to the full thread for interaction (reply, edit, delete etc...).
  • Comments will also show inline within the new (in-alpha) WYSIWYG editor when the comments sidebar is opened in the editor view.
  • To support this functionality, I've also added the ability to archive (and unarchive) comments, so that these kind of comments can be treated as a kind of todo-list, and archived when addressed.
    • Active/Archived threads are listed under separate tabs.
    • Inline comments will only be shown for the active comments tab.
  • Comments are marked if they have a reference.
    • This can be clicked to jump the the referenced section.
    • This will update to show as outdated if the reference section has changed.
      • This is based upon a hash of the text for that block section.
    • This will turn orange if the section no longer exists at all.
    • This will be gray in scenarios where we can't provide an interactive link or check the reference (in MD/old-wyswiyg editors)

Limitations

  • Only the top-level comment in a thread can include a reference (and also be archived/unarchived).
  • Only the new WYSIWYG editor receives inline support.
  • The inline comments are not interactive (no reply or other controls) but they do have the ability to jump to the full comment.
  • There's no display options/controls for these comments. Could be something that's needed but I thought we'd get the feature out there first to understand what level of control is actually needed (if at all).
  • The outdated/change detection is currently done browser-side, and is done block level (not just the text range if set).
    • I've tried to do this in a way we should be able to replicate on the PHP side if needed in the future.

Screenshots

Inline Display

Image

Inline Display - Opened

Image

Page View Comments List

Image

In Editor

Image

# for free to join this conversation on GitHub. Already have an account? # to comment
Development

No branches or pull requests