27920 2015-01-28 23:27:40 +0000 [WebDriver Spec]: takeScreenshot is a GET but what about screenshots of elements?? 2015-06-10 22:29:12 +0000 1 1 1 Unclassified Browser Test/Tools WG WebDriver unspecified PC All RESOLVED FIXED P2 normal --- 20860 1 dburns public-browser-tools-testing fisherii james johnjan mike vlotoshnikov public-browser-tools-testing oldest_to_newest 117559 0 dburns 2015-01-28 23:27:40 +0000 When ripping out webIDL I noticed that in takeScreenshot is a GET which aligns with the OSS Project implementation[1]. We also have the ability now to take screenshots of elements. This will either need to be a POST of the current url endpoint (yuck in my opinion) or a completely new endpoint. I propose we using GET /session/{sessionId}/screenshot/{ELEMENT} instead of changing the original call to POST. Thoughts? 117560 1 fisherii 2015-01-28 23:41:50 +0000 For consistency with other element-level operations I think: GET /session/{sessionId}/element/{element}/screenshot makes the most sense as an endpoint. 117570 2 james 2015-01-29 14:51:13 +0000 Yes, I think Marc is correct. 117571 3 dburns 2015-01-29 15:46:01 +0000 landed in 9cbf052 117572 4 dburns 2015-01-29 15:52:21 +0000 https://github.com/w3c/webdriver/commit/9cbf052d90a205115347f303b74fc26400bedbc5 118579 5 vlotoshnikov 2015-03-13 07:13:52 +0000 Spec text still says 'Leaving below until Bug 27920 has a consensus' 120893 6 johnjan 2015-06-10 22:06:02 +0000 I do think we have consensus on this, but the spec just needs to change to match it. Marc's proposal makes a lot of sense. 120894 7 james 2015-06-10 22:28:01 +0000 Apart from the fact that the spec text here needs some work, I don't think there is an actual issue. https://w3c.github.io/webdriver/webdriver-spec.html#element exists. 120895 8 james 2015-06-10 22:29:12 +0000 Oh unless you mean that the URL is still wrong. But it's just as quick to fix that as discuss it here; I'll make a PR.