Skip to content

Commit ffd3d62

Browse files
committed
rely on backend to control file-view-toggle-buttons visibility
1 parent 1d4ad5a commit ffd3d62

File tree

2 files changed

+8
-9
lines changed

2 files changed

+8
-9
lines changed

templates/repo/view_file.tmpl

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -36,12 +36,12 @@
3636
</div>
3737
<div class="file-header-right file-actions flex-text-block tw-flex-wrap">
3838
{{/* this componment is also controlled by frontend plugin renders */}}
39-
<div class="ui compact icon buttons file-view-toggle-buttons {{Iif .HasSourceRenderedToggle "" "tw-hidden"}}">
40-
{{if .IsRepresentableAsText}}
41-
<a href="?display=source" class="ui mini basic button file-view-toggle-source {{if .IsDisplayingSource}}active{{end}}" data-tooltip-content="{{ctx.Locale.Tr "repo.file_view_source"}}">{{svg "octicon-code" 15}}</a>
42-
{{end}}
43-
<a href="?display=rendered" class="ui mini basic button file-view-toggle-rendered {{if not .IsDisplayingSource}}active{{end}}" data-tooltip-content="{{ctx.Locale.Tr "repo.file_view_rendered"}}">{{svg "octicon-file" 15}}</a>
44-
</div>
39+
{{if and .HasSourceRenderedToggle .IsRepresentableAsText}}
40+
<div class="ui compact icon buttons file-view-toggle-buttons">
41+
<a href="?display=source" class="ui mini basic button file-view-toggle-source {{if .IsDisplayingSource}}active{{end}}" data-tooltip-content="{{ctx.Locale.Tr "repo.file_view_source"}}">{{svg "octicon-code" 15}}</a>
42+
<a href="?display=rendered" class="ui mini basic button file-view-toggle-rendered {{if not .IsDisplayingSource}}active{{end}}" data-tooltip-content="{{ctx.Locale.Tr "repo.file_view_rendered"}}">{{svg "octicon-file" 15}}</a>
43+
</div>
44+
{{end}}
4545
{{if not .ReadmeInList}}
4646
<div class="ui buttons tw-mr-1">
4747
<a class="ui mini basic button" href="{{$.RawFileLink}}">{{ctx.Locale.Tr "repo.file_raw"}}</a>

web_src/js/features/file-view.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import type {FileRenderPlugin} from '../render/plugin.ts';
22
import {newRenderPlugin3DViewer} from '../render/plugins/3d-viewer.ts';
33
import {newRenderPluginPdfViewer} from '../render/plugins/pdf-viewer.ts';
44
import {registerGlobalInitFunc} from '../modules/observer.ts';
5-
import {createElementFromHTML, showElem, toggleClass} from '../utils/dom.ts';
5+
import {createElementFromHTML, toggleClass} from '../utils/dom.ts';
66
import {html} from '../utils/html.ts';
77
import {basename} from '../utils.ts';
88

@@ -19,11 +19,10 @@ function findFileRenderPlugin(filename: string, mimeType: string): FileRenderPlu
1919

2020
function showRenderRawFileButton(elFileView: HTMLElement, renderContainer: HTMLElement | null): void {
2121
const toggleButtons = elFileView.querySelector('.file-view-toggle-buttons');
22-
showElem(toggleButtons);
22+
if (!toggleButtons) return;
2323
const displayingRendered = Boolean(renderContainer);
2424
toggleClass(toggleButtons.querySelectorAll('.file-view-toggle-source'), 'active', !displayingRendered); // it may not exist
2525
toggleClass(toggleButtons.querySelector('.file-view-toggle-rendered'), 'active', displayingRendered);
26-
// TODO: if there is only one button, hide it?
2726
}
2827

2928
async function renderRawFileToContainer(container: HTMLElement, rawFileLink: string, mimeType: string) {

0 commit comments

Comments
 (0)