Archives/gristlabs_grist-core
1
0
Fork 0
mirror of https://github.com/gristlabs/grist-core.git synced 2024-10-27 20:44:07 +00:00
Code Issues Projects Releases Wiki Activity

(core) Treat URLs in Markdown cells as absolute

Browse Source 
Summary: The previous behavior didn't match HyperLink cells.

Test Plan: Browser test.

Reviewers: jarek

Reviewed By: jarek

Subscribers: jarek

Differential Revision: https://phab.getgrist.com/D4358
This commit is contained in:
George Gevoian 2024-09-30 12:34:35 -04:00
parent d2714da224
commit 1927c87413
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/client/models/gristUrlState.ts
View File

@ -224,7 +224,7 @@ export class UrlStateImpl {
/**
* Given value like `foo bar baz`, constructs URL by checking if `baz` is a valid URL and,
* if not, prepending `http://`.
* if not, prepending `https://`.
*/
export function constructUrl(value: CellValue): string {
if (typeof value !== 'string') {
@ -235,8 +235,8 @@ export function constructUrl(value: CellValue): string {
// Try to construct a valid URL
return (new URL(url)).toString();
} catch (e) {
// Not a valid URL, so try to prefix it with http
return 'http://' + url;
// Not a valid URL, so try to prefix it with https
return 'https://' + url;
}
}

3
app/client/ui/MarkdownCellRenderer.ts
View File

@ -1,10 +1,11 @@
import {constructUrl} from 'app/client/models/gristUrlState';
import {gristIconLink} from 'app/client/ui2018/links';
import escape from 'lodash/escape';
import {marked} from 'marked';
export const renderer = new marked.Renderer();
renderer.link = ({href, text}) => gristIconLink(href, text).outerHTML;
renderer.link = ({href, text}) => gristIconLink(constructUrl(href), text).outerHTML;
// Disable Markdown features that we aren't ready to support yet.
renderer.hr = ({raw}) => raw;