- Issue created by @ressa
- 🇩🇰Denmark ressa Copenhagen
I have now switched to using the module as intended, using terms ... but maybe this is still a useful feature?
- Status changed to Needs review
over 1 year ago 7:01pm 8 May 2023 - 🇩🇰Denmark ressa Copenhagen
Actually, I now see that I could definitely use this feature, doing a search on a term, instead of linking to the term itself, so changing Status.
- last update
over 1 year ago 15 pass - last update
over 1 year ago 15 pass - Status changed to Active
over 1 year ago 7:17pm 8 May 2023 - 🇩🇰Denmark ressa Copenhagen
Hmm, okay. I guess I need to update branch from 2.0.x to 3.x both here and on Gitlab.
The "Run test/retest"-options have disappeared, after changing branch on Gitlab and rebasing, though ... Maybe I'll have to change Status to "Active" and back to "Needs review" to trigger testing against 3.0?
- Status changed to Needs review
over 1 year ago 7:18pm 8 May 2023 - 🇩🇰Denmark ressa Copenhagen
Fingers crossed, tests will appear after this ...
- 🇩🇰Denmark ressa Copenhagen
Could it simply be because tests have not been enabled for 3.x branch?
Hello Ressa
I'm following up from the pointer you left on this issue:
https://www.drupal.org/project/glossify/issues/3351469 ✨ merge with Word Link ActiveI have installed your patch, and now I'm trying to figure out how to use Glossify.
I activated the links and tooltips with taxonomy.
Now I'm trying to figure out how to make it work for my purpose, for example, to link every occurrence of Google in a Text be linked to "https://google.com"
- 🇩🇰Denmark ressa Copenhagen
I tried enabling "Glossify: Tooltips with taxonomy" under /admin/config/content/formats/manage/basic_html and added
https://example.org?search=[name]
but get this error:InvalidArgumentException: The internal path component 'https://example.org' is invalid. Its path component must have a leading slash, e.g. internal:/foo. in Drupal\Core\Url::fromInternalUri() (line 416 of core/lib/Drupal/Core/Url.php).
So it looks like only internal links work.
Yes, that is what I understood too.
Fortunately, an older project I already had dismissed as being abandoned has released a D9 version some time last year (I missed it) and now there's a D10 version too:
https://www.drupal.org/project/alinks →I haven't tested it yet, but it looks promising
- 🇩🇰Denmark ressa Copenhagen
That other project looks stalled. I think a better approach would be to create an issue here, with a feature request to allow external links.
well, I already requested a merge with "word link" which is essentially a clone of alinks.
alinks had a new dev release 7th September 2023.
- Status changed to Needs work
7 months ago 1:48am 10 June 2024 - 🇳🇿New Zealand klidifia
The added lines are overwriting the variable declared in the previous line, so whilst this lets [name] be used, it means [id] cannot be used.
Lets see if we can make them both work.
- Status changed to Needs review
7 months ago 2:06am 10 June 2024 - 🇳🇿New Zealand klidifia
Have put both replacements in the same line, I can utilise both tokens OK.
- Status changed to Needs work
5 months ago 3:32pm 26 July 2024 - 🇩🇪Germany Anybody Porta Westfalica
@ressa thank you very much for the idea. I first thought, what you describe should better be a separate module or a submodule, but essentially your implementation just adds a further replacement.
Now I think it would be even better to implement token support here to unify the functionality instead of adding further custom syntax / pseudo-tokens. What do you think about that?
Could you do it? - 🇩🇰Denmark ressa Copenhagen
Thanks for looking at this @Anybody and @klidifia for updating the patch. It's a great idea to add Token support for even more flexibility -- maybe @klidifia is able to add this feature?