"Linked to source map" file path is incorrect when the folders have dashes in them.
Reported by
m...@codeavengers.com,
Mar 8 2017
|
||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3033.0 Safari/537.36 Steps to reproduce the problem: 1. Turn on dev tools experiment Persistence 2.0. 2. Store less files in a folder called less-modules 3. When the auto mapping happens, it says "Linked to source map: http://localhost/less/-modules/file.less" What is the expected behavior? It should be: http://localhost/less-modules/file.less What went wrong? Some files in that folder are automatically linked, others don't automatically link. Also, sometimes when I click through to less files from the Elements CSS inspector, the less file shows up blank. Did this work before? N/A Chrome version: 59.0.3033.0 Channel: canary OS Version: 10.0 Flash Version:
,
Mar 9 2017
mike@codeavengers@ could you please provide a sample test file or url to reproduce the issue and possible help us by providing the screen cast of the issue to triaging it better. Thanks.
,
Mar 10 2017
The linking doesn't work for less files or javascript files that are inside a directory with a dash in the name. For example I have a javascript directory called markup-plugins/js and none of the files are automatically linked. It seems the system is assuming that folders do NOT contain dashes in them.
,
Mar 10 2017
Thank you for providing more feedback. Adding requester "sureshkumari@chromium.org" to the cc list and removing "Needs-Feedback" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 10 2017
I'll do a screen cast to show you an example of it happening... because it does work sometimes...
,
Mar 14 2017
mike@ could you please provide the screen-cast as per comment #5, for further triage. Thank You...
,
Mar 21 2017
I am having trouble reproducing this. @lushnikov do you know what is going on?
,
Mar 21 2017
Sorry about that. I'll do screen cast...
,
Mar 21 2017
Thank you for providing more feedback. Adding requester "kkaluri@chromium.org" to the cc list and removing "Needs-Feedback" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 21 2017
Hi Mike! Could you please share with us a screenshot with you Sources navigator? Both "filesystem" and "network" tabs.
,
Mar 21 2017
These unfortunately don't show up; could you please attach them via web-interface?
,
Mar 21 2017
OK...let's try this again... last time some bot automatically deleted my comment.
,
Mar 22 2017
Nice, this is helpful! Any chance you can attach the .css.map file?
,
Mar 22 2017
,
Mar 28 2017
Latest releases have made things worse. Now I can't even get the mappings working at all. https://www.screencast.com/t/LsTIL7cpOnN
,
Mar 28 2017
Hi Mike, Which tool do you use to generate the sourcemap file? It is generated incorrectly - it has broken source urls (see screenshot)
,
Mar 28 2017
Ah ok! So it is a problem at our end. I'll check with our team member that takes care of that.
,
Mar 28 2017
I just checked the latest version of our source maps and they seem to be correct now. Perhaps the mappings have gotten thrown out by them being incorrect previously. I think we updated to a more recent version of the less compiler this week which has fixed a few bugs with source maps.
,
Mar 29 2017
It seems the problem was a bug with the old version of the less compiler. This bug is fixed in the new less compiler.
,
Mar 29 2017
Good to hear, closing this then.
,
Mar 29 2017
OK We still haven't quite got things sorted... my developer is still trying to find which combination of less compiler parameters yields the correct output that dev tools understands. At present he's spitting out fullpaths as follows for a less file that is available at localhost://markup-plugins/less/grid-match.less : ["C:/Users/Public/workspace/CodeAvengers/war/less-modules/variables.less","C:/Users/Public/workspace/CodeAvengers/war/markup-plugins/less/grid-match.less"] We previously used Koala to compile our less. Its source map works with dev tools. It spits out the following paths: "..\\..\\..\\..\\..\\..\\..\\Program Files (x86)\\less\\C:\\Users\\Public\\workspace\\CodeAvengers\\war\\less-modules\\variables.less","../less/grid-match.less" Why does one of these work and the other doesn't? Shouldn't both of these work? What are the settings to achieve this final outcome?
,
Mar 29 2017
#27: DevTools expect sources to be URLs (possibly relative urls), not paths. URLs are platform-independent, but paths aren't. In your examples: - "C:/Users/Public/workspace/CodeAvengers/war/less-modules/variables.less" is a file path rather then URL, it has forward slashes and couldn't be parsed as URL. DevTools fails to handle this properly - "..\\..\\..\\..\\..\\..\\..\\Program Files (x86)\\less\\C:\\Users\\Public\\workspace\\CodeAvengers\\war\\less-modules\\variables.less" could be treated as a relative URL, and for this reason it probably works.
,
Mar 29 2017
Thanks that makes sense. Mike |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by ajha@chromium.org
, Mar 9 2017