mirror of
https://codeberg.org/forgejo/forgejo
synced 2024-12-05 02:54:46 +01:00
df74ee0376
Backport #23702 by @jpraet
Fixes #23701, #23515.
Alternate approach to #23604 using CSS scroll-margin-top, which is also
taken into account for direct links to files in a diff:
* On the PR diff, this currently shows the previous file first:
https://try.gitea.io/jpraet/test/pulls/13/files#diff-b94d08b409f9d05fb65b6cccaf7b3e4ecc7cc333
* On the commit diff, the first line of the linked file is currently
under the sticky header:
1a19e6b14e (diff-b94d08b409f9d05fb65b6cccaf7b3e4ecc7cc333)
Co-authored-by: Jimmy Praet <jimmy.praet@ksz-bcss.fgov.be>
20 lines
995 B
JavaScript
20 lines
995 B
JavaScript
import {svg} from '../svg.js';
|
|
|
|
// Hides the file if newFold is true, and shows it otherwise. The actual hiding is performed using CSS.
|
|
//
|
|
// The fold arrow is the icon displayed on the upper left of the file box, especially intended for components having the 'fold-file' class.
|
|
// The file content box is the box that should be hidden or shown, especially intended for components having the 'file-content' class.
|
|
//
|
|
export function setFileFolding(fileContentBox, foldArrow, newFold) {
|
|
foldArrow.innerHTML = svg(`octicon-chevron-${newFold ? 'right' : 'down'}`, 18);
|
|
fileContentBox.setAttribute('data-folded', newFold);
|
|
if (newFold && fileContentBox.getBoundingClientRect().top < 0) {
|
|
fileContentBox.scrollIntoView();
|
|
}
|
|
}
|
|
|
|
// Like `setFileFolding`, except that it automatically inverts the current file folding state.
|
|
export function invertFileFolding(fileContentBox, foldArrow) {
|
|
setFileFolding(fileContentBox, foldArrow, fileContentBox.getAttribute('data-folded') !== 'true');
|
|
}
|
|
|