Clipper: Fixes #1990: Clipper was not saving page URL when taking scr…

…eenshot
laurent22 · Nov 7, 2019 · 4af8bb8 · 4af8bb8
1 parent a6f1907
commit 4af8bb8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/Clipper/joplin-webclipper/background.js b/Clipper/joplin-webclipper/background.js
@@ -57,6 +57,7 @@ browser_.runtime.onMessage.addListener(async (command) => {
 		const imageDataUrl = await browserCaptureVisibleTabs(null);
 		const content = Object.assign({}, command.content);
 		content.image_data_url = imageDataUrl;
+		if ('url' in content) content.source_url = content.url;
 
 		const newArea = Object.assign({}, command.content.crop_rect);
 		newArea.x *= zoom;

diff --git a/Clipper/joplin-webclipper/content_scripts/index.js b/Clipper/joplin-webclipper/content_scripts/index.js
@@ -433,7 +433,7 @@
 					const content = {
 						title: pageTitle(),
 						crop_rect: selectionArea,
-						url: pageLocationOrigin() + location.pathname,
+						url: pageLocationOrigin() + location.pathname + location.search,
 						parent_id: command.parent_id,
 						tags: command.tags,
 					};