mirror of
https://github.com/go-gitea/gitea.git
synced 2026-08-27 06:03:42 +00:00
2bcf950b78
Adds a search box and a file-extension filter to the pull request diff sidebar, so reviewers can narrow a large diff down to the files they care about. Both filters apply to the file tree and to the diff itself. The extension menu follows GitHub: extensions sorted alphabetically, dotfiles and extension-less files in their own buckets, and the selection kept in the same `file-filters[]` query parameter, so a filtered view is shareable and survives a reload. The menu can list every extension in a diff, so `createTippy` gains an opt-in `limitSizeToViewport` option that caps a popup to the space left in the viewport and scrolls its content. Popups that do not ask for it are unchanged. Closes https://github.com/go-gitea/gitea/issues/27256 Signed-off-by: silverwind <me@silverwind.io> Signed-off-by: wxiaoguang <wxiaoguang@gmail.com> Co-authored-by: silverwind <me@silverwind.io> Co-authored-by: Claude (Opus 4.7) <noreply@anthropic.com> Co-authored-by: Copilot <copilot@github.com> Co-authored-by: Nicolas <bircni@icloud.com> Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
24 lines
1.0 KiB
TypeScript
24 lines
1.0 KiB
TypeScript
import DiffFileTreeItem from './DiffFileTreeItem.vue';
|
|
import {createApp, h} from 'vue';
|
|
import type {DiffStatus, DiffTreeEntry} from '../modules/diff-file.ts';
|
|
|
|
function renderItem(diffStatus: string): string {
|
|
const item: DiffTreeEntry = {
|
|
FullName: 'a.txt', OldFullName: '', DisplayName: 'a.txt', NameHash: 'hash',
|
|
DiffStatus: diffStatus as DiffStatus, EntryMode: '', IsViewed: false, Children: null, FileIcon: '',
|
|
};
|
|
const root = document.createElement('div');
|
|
createApp({render: () => h(DiffFileTreeItem, {item})}).mount(root);
|
|
return root.innerHTML;
|
|
}
|
|
|
|
test('DiffFileTreeItem diff status icon', () => {
|
|
window.config.pageData.DiffFileTree = {TreeRoot: {
|
|
FullName: '', OldFullName: '', DisplayName: '', NameHash: 'root',
|
|
DiffStatus: '', EntryMode: 'tree', IsViewed: false, Children: [], FileIcon: '',
|
|
}};
|
|
expect(renderItem('typechanged')).toContain('octicon-diff-modified');
|
|
// a status the frontend does not know must fall back instead of failing to render
|
|
expect(renderItem('something-new')).toContain('octicon-blocked');
|
|
});
|