mirrors/zotero
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Set a default annotation color if not assigned

Browse source 
https://forums.zotero.org/discussion/104249/sync-issue-on-ipados
This commit is contained in:
Dan Stillman 2023-04-09 16:58:51 -04:00
parent b33d3ec799
commit cadfca7475
3 changed files with 19 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
chrome/content/zotero/xpcom/annotations.js
View file

@ -33,6 +33,8 @@ Zotero.Annotations = new function () {
Zotero.defineProperty(this, 'ANNOTATION_TYPE_IMAGE', { value: 3 });
Zotero.defineProperty(this, 'ANNOTATION_TYPE_INK', { value: 4 });
Zotero.defineProperty(this, 'DEFAULT_COLOR', { value: '#ffd400' });
Zotero.defineProperty(this, 'PROPS', {
value: ['type', 'authorName', 'text', 'comment', 'color', 'pageLabel', 'sortIndex', 'position'],
writable: false

2
chrome/content/zotero/xpcom/data/item.js
View file

@ -1935,7 +1935,7 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
authorName || null,
text || null,
comment || null,
color || null,
color || Zotero.Annotations.DEFAULT_COLOR,
pageLabel || null,
sortIndex,
position,

16
test/tests/itemTest.js
View file

@ -1400,6 +1400,22 @@ describe("Zotero.Item", function () {
assert.isFalse(annotation.hasChanged());
});
it("should assign a default color", async function () {
var annotation = new Zotero.Item('annotation');
annotation.parentID = attachment.id;
annotation.annotationType = 'highlight';
annotation.annotationText = "This is highlighted text.";
annotation.annotationSortIndex = '00015|002431|00000';
annotation.annotationPosition = JSON.stringify({
pageIndex: 123,
rects: [
[314.4, 412.8, 556.2, 609.6]
]
});
await annotation.saveTx();
assert.equal(annotation.annotationColor, '#ffd400');
});
it("should save a note annotation", async function () {
var annotation = new Zotero.Item('annotation');
annotation.parentID = attachment.id;