68092082f2
* Enable access to old notetype name * Set minimum height for ChangeNotetypeDialog * Add bootstrap icons to change-notetype * Move alert up and make it collapsible * Tweak some CSS - Add variables --sticky-bg and --sticky-border to StickyContainer - Tweak base.css * Add translatable string "(Nothing)" * Rework ChangeNotetype screen * Initially load option at newIndex and remaining options on focus Optimization for big notetypes: Should increase efficiency from O(n²) to O(n). Test on notetype with 500 templates shows significant improvement in load time (~10s down to ~1s). * Try to satisfy rust test * Change arrow direction depending on reading direction + add 0.5em top padding to main * Create Alert.svelte * Introduce CSS variable --pane-bg * Revert "Initially load option at newIndex and remaining options on focus" This reverts commit f42beee45c27dba9433d76217fb583b117fb5231. * Final cleanup * Refine padding/gutter
130 lines
4.1 KiB
TypeScript
130 lines
4.1 KiB
TypeScript
// Copyright: Ankitects Pty Ltd and contributors
|
|
// License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
|
|
|
|
/* eslint
|
|
@typescript-eslint/no-explicit-any: "off",
|
|
*/
|
|
|
|
import { ChangeNotetypeState, negativeOneToNull, MapContext } from "./lib";
|
|
import { Notetypes } from "../lib/proto";
|
|
import { get } from "svelte/store";
|
|
import * as tr from "../lib/ftl";
|
|
|
|
const exampleNames = {
|
|
entries: [
|
|
{
|
|
id: "1623289129847",
|
|
name: "Basic",
|
|
},
|
|
{
|
|
id: "1623289129848",
|
|
name: "Basic (and reversed card)",
|
|
},
|
|
{
|
|
id: "1623289129849",
|
|
name: "Basic (optional reversed card)",
|
|
},
|
|
{
|
|
id: "1623289129850",
|
|
name: "Basic (type in the answer)",
|
|
},
|
|
{
|
|
id: "1623289129851",
|
|
name: "Cloze",
|
|
},
|
|
],
|
|
};
|
|
|
|
const exampleInfoDifferent = {
|
|
oldFieldNames: ["Front", "Back"],
|
|
oldTemplateNames: ["Card 1"],
|
|
newFieldNames: ["Front", "Back", "Add Reverse"],
|
|
newTemplateNames: ["Card 1", "Card 2"],
|
|
input: {
|
|
newFields: [0, 1, -1],
|
|
newTemplates: [0, -1],
|
|
oldNotetypeId: "1623289129847",
|
|
newNotetypeId: "1623289129849",
|
|
currentSchema: "1623302002316",
|
|
oldNotetypeName: "Basic",
|
|
},
|
|
};
|
|
|
|
const exampleInfoSame = {
|
|
oldFieldNames: ["Front", "Back"],
|
|
oldTemplateNames: ["Card 1"],
|
|
newFieldNames: ["Front", "Back"],
|
|
newTemplateNames: ["Card 1"],
|
|
input: {
|
|
newFields: [0, 1],
|
|
newTemplates: [0],
|
|
oldNotetypeId: "1623289129847",
|
|
newNotetypeId: "1623289129847",
|
|
currentSchema: "1623302002316",
|
|
oldNotetypeName: "Basic",
|
|
},
|
|
};
|
|
|
|
function differentState(): ChangeNotetypeState {
|
|
return new ChangeNotetypeState(
|
|
Notetypes.NotetypeNames.fromObject(exampleNames),
|
|
Notetypes.ChangeNotetypeInfo.fromObject(exampleInfoDifferent),
|
|
);
|
|
}
|
|
|
|
function sameState(): ChangeNotetypeState {
|
|
return new ChangeNotetypeState(
|
|
Notetypes.NotetypeNames.fromObject(exampleNames),
|
|
Notetypes.ChangeNotetypeInfo.fromObject(exampleInfoSame),
|
|
);
|
|
}
|
|
|
|
test("proto conversion", () => {
|
|
const state = differentState();
|
|
expect(get(state.info).fields).toStrictEqual([0, 1, null]);
|
|
expect(negativeOneToNull(state.dataForSaving().newFields)).toStrictEqual([
|
|
0,
|
|
1,
|
|
null,
|
|
]);
|
|
});
|
|
|
|
test("mapping", () => {
|
|
const state = differentState();
|
|
expect(get(state.info).getNewName(MapContext.Field, 0)).toBe("Front");
|
|
expect(get(state.info).getNewName(MapContext.Field, 1)).toBe("Back");
|
|
expect(get(state.info).getNewName(MapContext.Field, 2)).toBe("Add Reverse");
|
|
expect(get(state.info).getOldNamesIncludingNothing(MapContext.Field)).toStrictEqual(
|
|
["Front", "Back", tr.changeNotetypeNothing()],
|
|
);
|
|
expect(get(state.info).getOldIndex(MapContext.Field, 0)).toBe(0);
|
|
expect(get(state.info).getOldIndex(MapContext.Field, 1)).toBe(1);
|
|
expect(get(state.info).getOldIndex(MapContext.Field, 2)).toBe(2);
|
|
state.setOldIndex(MapContext.Field, 2, 0);
|
|
expect(get(state.info).getOldIndex(MapContext.Field, 2)).toBe(0);
|
|
|
|
// the same template shouldn't be mappable twice
|
|
expect(
|
|
get(state.info).getOldNamesIncludingNothing(MapContext.Template),
|
|
).toStrictEqual(["Card 1", tr.changeNotetypeNothing()]);
|
|
expect(get(state.info).getOldIndex(MapContext.Template, 0)).toBe(0);
|
|
expect(get(state.info).getOldIndex(MapContext.Template, 1)).toBe(1);
|
|
state.setOldIndex(MapContext.Template, 1, 0);
|
|
expect(get(state.info).getOldIndex(MapContext.Template, 0)).toBe(1);
|
|
expect(get(state.info).getOldIndex(MapContext.Template, 1)).toBe(0);
|
|
});
|
|
|
|
test("unused", () => {
|
|
const state = differentState();
|
|
expect(get(state.info).unusedItems(MapContext.Field)).toStrictEqual([]);
|
|
state.setOldIndex(MapContext.Field, 0, 2);
|
|
expect(get(state.info).unusedItems(MapContext.Field)).toStrictEqual(["Front"]);
|
|
});
|
|
|
|
test("unchanged", () => {
|
|
let state = differentState();
|
|
expect(get(state.info).unchanged()).toBe(false);
|
|
state = sameState();
|
|
expect(get(state.info).unchanged()).toBe(true);
|
|
});
|