mirror of
https://codeberg.org/forgejo/forgejo
synced 2025-10-19 15:00:54 +02:00
c8ec9bc2cc
Some Vue components were already lazy-loaded, but not all. There are three main changes: 1. Move init scripts outside Vue code. 2. Make the init scripts lazy-load the Vue components. 3. Deal with vue linters, because apparently if you do `export default` then some Vue linters will notice that's vue sfc. The rationale for this pull requests is that this reduces the size of the `index.js` file. - Uncompressed: 1.24 MB -> 954.40 kB - Compressed (gzip, via `reverse_proxy` of Caddy): 400.45 kB -> 298.95 kB - Compressed (zstd, via `bindata` tag): 363.75 kB -> 274.22 kB Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/9444 Reviewed-by: Otto <otto@codeberg.org> Co-authored-by: Gusted <postmaster@gusted.xyz> Co-committed-by: Gusted <postmaster@gusted.xyz>
43 lines
1.2 KiB
JavaScript
43 lines
1.2 KiB
JavaScript
import {createApp} from 'vue';
|
|
import {parseIssueHref} from '../utils.js';
|
|
import {createTippy} from '../modules/tippy.js';
|
|
|
|
export function initContextPopups() {
|
|
const refIssues = document.querySelectorAll('.ref-issue');
|
|
attachRefIssueContextPopup(refIssues);
|
|
}
|
|
|
|
export async function attachRefIssueContextPopup(refIssues) {
|
|
for (const refIssue of refIssues) {
|
|
if (refIssue.classList.contains('ref-external-issue')) {
|
|
return;
|
|
}
|
|
|
|
const {owner, repo, index} = parseIssueHref(refIssue.getAttribute('href'));
|
|
if (!owner) return;
|
|
|
|
const el = document.createElement('div');
|
|
refIssue.parentNode.insertBefore(el, refIssue.nextSibling);
|
|
|
|
const {default: ContextPopup} = await import(/* webpackChunkName: "context-popup" */'../components/ContextPopup.vue');
|
|
const view = createApp(ContextPopup);
|
|
|
|
try {
|
|
view.mount(el);
|
|
} catch (err) {
|
|
console.error(err);
|
|
el.textContent = 'ContextPopup failed to load';
|
|
}
|
|
|
|
createTippy(refIssue, {
|
|
content: el,
|
|
placement: 'top-start',
|
|
interactive: true,
|
|
role: 'dialog',
|
|
interactiveBorder: 5,
|
|
onShow: () => {
|
|
el.firstChild.dispatchEvent(new CustomEvent('ce-load-context-popup', {detail: {owner, repo, index}}));
|
|
},
|
|
});
|
|
}
|
|
}
|