miguelfreitas / twister-html

twister HTML + Javascript User Interface
MIT License
229 stars 138 forks source link

Link to a message in twister #120

Open idontsov opened 10 years ago

idontsov commented 10 years ago

Suggestion: introduce a common way to reference a message in twister. A client application should display such references as hyperlinks.

iShift commented 10 years ago

+1 we need wrapper that can open twister:// links

dryabov commented 10 years ago

IMHO it's better to use just "twister:" (without slashes), like in mailto, skype, etc.:

twister:messageid@username
iShift commented 10 years ago

show message: twister:show:messageid@username show username twister:show:username follow: twister:follow:@username send post: twister:send:%text% send DM: twister:dm:@username

dryabov commented 10 years ago

@iShift Why do you suggest so many links? IMHO, it would be enough to have links to posts twister:messageid@username and to profiles twister:username (where follow, post, and DM buttons are displayed). The rest is hardly necessary. Can you give an example where it would be used?

iShift commented 10 years ago

Twitter widgets on many sites "follow me, follow us" with one click following

milouse commented 9 years ago

I've started working on a way to directly open various modal window, following the url hash: milouse/twister-html@7ed9b8b84f3d850374b86100908fdb32721d7e13. Current state juste allow to directly open user profile when opening webpage like http://yourhost/home.html#profile?user=toto I need to polish it and monitor hashtag and direct link to message if possible. Stay tuned.

milouse commented 9 years ago

I've successfully implemented external url support and use of web+twister:profile:username and web+twister:hashtag:something protocol handler. You can review it in my branch.

I need more support to understand how to retrieve only one message in order to implement web+twister:profile:username:messageid If someone has some hints, it would be great :+1:

miguelfreitas commented 9 years ago

@milouse I'm backlogged and i still haven't had time to check your contributions. Still, thanks for you work in advance ;-)

I believe there are two kind of orthogonal issues here:

1) when @idontsov opened this issue #120, i believe he was referring to how to address old posts from inside the post text itself. for that purpose, i like @dryabov suggestion of short urls.

2) the ability to open http://localhost:28332/home.html#profile?user=xxx or similar is very nice (and quite important for browser navigation, that is, issue #199) but it's not necessary for (1).

i mean, one could resolve (1) without touching (2) or vice versa.

milouse commented 9 years ago

Yep, I totally agree with that. In fact one can use my work to address 2) without having to think about 1) (or vice versa). I merge the two concepts in my PR because I thought, from a developer point of view, it makes sense and avoid similar code duplication. That's why I introduce the function watchHashChange (https://github.com/milouse/twister-html/blob/feature/followhashlinks/js/interface_common.js#L356) to monitor both « real » or tiny short url and take appropriate action.

But yes, my work is not complete. This function must be extended to fully address #120. But I lack of knowledge in C++/DHT/torrent and other twister-core stuff xD So for now development is currently postponed for this feature, until I found some times to further investigate inside core functions.

By the way, I think my PR should be usefull to close #41 and no problem to volonteer on closing this #120 issue too :)