Link Preview

datlechin

clarkwinkelmann sorry about that, idk
This is just temporary using
I'll switch to using PHP or JS library to get data soon
As for proxies, I don't know about this, but I'll try to find out

010101

clarkwinkelmann I see Google servers are used to serve the favicons.

@datlechin an idea 💡 I know it would not be as “cool” but you could just remove favicons. Then you aren’t having to use that Google service. Still, one day, like you said, also using something else to get the meta data would be best. But, either way, not worrying about favicons keeps it more simple.

datlechin

I'm have updated external services used for now

datlechin

v0.1.5 @6bb55bf, v0.1.6 @d9680ae

Using PHP library to fetch open grap instead of a third-party app.

girefko

datlechin Any idea how to make the below link more visible?

datlechin

girefko Use Custom CSS

.LinkPreview-domain a {
    color: // put your color
}

datlechin

v0.1.7 datlechin/flarum-link-preview`afee215`

Now the title will show "This site cannot be reached" instead of returning an exception when the domain name is cannot reached.

datlechin

010101 if a website goes down

datlechin you mean this?

010101

Haven’t had time to install and try this yet. How is it on page load performance? I assume it has to ping the website and get the open graph data each time? Because I think the only way around that is some sort of caching or database storing just for the meta data.

datlechin

010101 How is it on page load performance

I'm not sure about this, it depends on your server.

010101 Because

That's a good idea, but I don't know how to do it right now

010101

datlechin it would be difficult, I think. One option would be, as you post, it grabs the meta og image, and uploads it somewhere. Like the upload extension does. You then at least have the link’s image stored. But I don’t know how. That is still above my programming knowledge. 😳

datlechin

010101 what if store them on localstorage 🤔

010101

datlechin

I fear, then too much ends up being stored locally. And I think people would need to add to their privacy polices that this site stores images locally.

datlechin

010101 Just store the link image, title, and description of each link..... I think storing them in session storage would be better. Because after closing the browser, the data stored on it will be destroyed.

010101

datlechin Oh ok. Yes, this would probably help.

I just assume the big sites (Facebook/Twitter), they somehow cache not just the link to the image, but the image itself. This way, every time you load the page it’s not asking that website for its open graph image.

But, some kind of local storage could be one step in the right direction.

Another question: How’s this working for inline links? For example I may not want this link turned into a preview.

Umutcan

In this way, it is less tiring on the eyes.

/* Link Preview */
.LinkPreview {
background: @hero-bg !important;
}
.LinkPreview-description {
color: @muted-color !important;
}
.LinkPreview a {
color: @muted-color !important;
}
.LinkPreview .fa-spinner{
color: @muted-color !important;
}
/* Link Preview End */

[deleted]

Umutcan
Thanks for sharing this code...

datlechin

010101 How’s this working for inline links? For example I may not want this link turned into a preview.

It works for this way. create links using markdown

010101

datlechin thanks I plan on trying it out today or tonight. Another argument for somehow saving or caching what the meta grabber gets is if a website goes down, then your link preview is broken. I wish I knew how to do it. Still this seems like the closest Flarum has gotten in a long time to a link preview extension that’s free.

datlechin

V0.1.8 datlechin/flarum-link-preview`9de53e6`

Now you can enter blacklist urls or domains to exclude from link preview.
Larger and fixed image on top of Link preview on tablets and desktops.

010101

datlechin

How hard is it to add a whitelist box instead? Only allow a few URLs.

ctml

datlechin

Great work! My first thought when I saw this thread was it would be nice to exclude local domains for intranet sites as I don't want external services to learn about my internal domains... Read through all the posts and sure enough you added this blacklist feature 🤯

Does the whole domain need to match, or only partial?

E.g. would site1.company.com be ignored if company.com was in the ignore list, if not would it be possible to add that behavior or alternatively support regex?

« Previous Page Next Page »