-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add "a href" links to clickable areas for messages and profiles and chats #81
Comments
I don't think it's simple to do that. In order to (easily) make right-click work to open something, it needs to be inside an There are probably ways to make this work, e.g. a custom right-click menu, but I think it's too much effort for the likely use of this case. That's partly a decision about the time we have available for making changes, so if someone else wanted to implement it, I'd be happy to review and merge it. Similar things apply in other cases, but it might be easy to do for some cases where there isn't a complex DOM structure. I can look at them if you pick one or two favourites. |
Thanks for the reply. Yeah, I wondered if the linked group name might be a problem. I don't think a custom right-click menu is a good idea. Personally, I'd be OK with the group name not being clickable when the messages are shown in a list as in the screenshots - i.e. the group link could only be clickable once the user is viewing the full message (whether in a popup or in a new tab). |
Sure, but there might be other examples of this. So we'd have to search for them to make sure, and we'd also be vulnerable to bugs being introduced in future by inserting I do agree with the aim, but I think it's too expensive for me to implement given the constraints on my time and the likely usage. Sorry. I'll leave the issue open for a bit in case someone else wanders along and decides to look at it, though, or comes up with a cheap solution I've not thought of. |
I understand :) |
This is a continuation of the issue I wrongly filed at Freegle/iznik-nuxt#416
You have kindly added "a href" links to the title of a message, which is great, but I think this could be expanded.
At the moment, a normal click or tap anywhere in the following area shows the message in an overlay popup:
![image](https://private-user-images.githubusercontent.com/2512915/311032966-0d121fb9-edb5-46bb-b164-935e60e1a3b0.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjQ3NjY1ODgsIm5iZiI6MTcyNDc2NjI4OCwicGF0aCI6Ii8yNTEyOTE1LzMxMTAzMjk2Ni0wZDEyMWZiOS1lZGI1LTQ2YmItYjE2NC05MzVlNjBlMWEzYjAucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDgyNyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA4MjdUMTM0NDQ4WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9NzZlYmYzNGJiN2EyMzU0ZGYwNDUzMzBkZjAzMGQzZmZiN2ZhYjEyODQ0ODdiZGNiZGNlMDEwOTU0ZTJjM2M4YSZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.YdsXT3_GfVl_KpWNMPj4tgefFFVIcqMbi8v0OJExor8)
With 037a76d a ctrl-click (or right-click or long-press and "open in new tab") on the following area opens the message in a new tab:
I think it would be better if a ctrl-click (etc.) anywhere in the larger area in the first image would open in a new tab, i.e. if all that area was inside the "a href", if that's possible. In other words, the same area that opens up the message in a popup should also be available to use to opening in a new tab if the user desires.
In the original issue, I also mentioned it would be good to "linkify" profiles too, so that ctrl-clicking on the following area:
![image](https://private-user-images.githubusercontent.com/2512915/311033873-0517d636-e739-49aa-90bd-3bcd5c5ecace.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjQ3NjY1ODgsIm5iZiI6MTcyNDc2NjI4OCwicGF0aCI6Ii8yNTEyOTE1LzMxMTAzMzg3My0wNTE3ZDYzNi1lNzM5LTQ5YWEtOTBiZC0zYmNkNWM1ZWNhY2UucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDgyNyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA4MjdUMTM0NDQ4WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9NWZkNjA5ZGI0NDRlODQxZDMyMWU2NjRjYWYwMDgzYTMwM2I5MWYwMGUzNzUzYTk0NjA4MDRkNWQ1MWIwYjJkNSZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.pAhTldMJ8w3Hn_VeoP3GTRavk7t1o6H8Y0aMWlFChFk)
would open a new tab to ilovefreegle.org/profile/xxxxxxxx.
While viewing a profile page, it would be good if a link could be added to the Message button (to a ilovefreegle.org/chats/xxxxxxx URL), so the user could ctrl-click (etc.) that too, to open in a new tab.
While viewing any ilovefreegle.org/chats page, it would be good if links could be added to each individual chat entry in the left-hand pane, so they could be ctrl-clicked (etc.) to open each chat in a new tab.
There's probably other examples that I've missed, but I hope you get the idea! I.e. anywhere that is normally clickable and opens a popup or which changes the URL, should have links on them so the user can open in a new tab if they desire.
Sorry if this is long-winded, hope this is OK.
The text was updated successfully, but these errors were encountered: