{"action":"created","issue":{"url":"https://api.github.com/repos/fontforge/fontforge/issues/1803","labels_url":"https://api.github.com/repos/fontforge/fontforge/issues/1803/labels{/name}","comments_url":"https://api.github.com/repos/fontforge/fontforge/issues/1803/comments","events_url":"https://api.github.com/repos/fontforge/fontforge/issues/1803/events","html_url":"https://github.com/fontforge/fontforge/issues/1803","id":45936584,"number":1803,"title":"2014-10-14 Win7 Crash/confusion in initial Open font dialog","user":{"login":"tshinnic","id":842881,"avatar_url":"https://avatars.githubusercontent.com/u/842881?v=3","gravatar_id":"","url":"https://api.github.com/users/tshinnic","html_url":"https://github.com/tshinnic","followers_url":"https://api.github.com/users/tshinnic/followers","following_url":"https://api.github.com/users/tshinnic/following{/other_user}","gists_url":"https://api.github.com/users/tshinnic/gists{/gist_id}","starred_url":"https://api.github.com/users/tshinnic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tshinnic/subscriptions","organizations_url":"https://api.github.com/users/tshinnic/orgs","repos_url":"https://api.github.com/users/tshinnic/repos","events_url":"https://api.github.com/users/tshinnic/events{/privacy}","received_events_url":"https://api.github.com/users/tshinnic/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/fontforge/fontforge/labels/I-bug","name":"I-bug","color":"fc2929"},{"url":"https://api.github.com/repos/fontforge/fontforge/labels/I-cleanup","name":"I-cleanup","color":"e11d21"},{"url":"https://api.github.com/repos/fontforge/fontforge/labels/I-completion","name":"I-completion","color":"e11d21"},{"url":"https://api.github.com/repos/fontforge/fontforge/labels/I-crash","name":"I-crash","color":"e10c02"},{"url":"https://api.github.com/repos/fontforge/fontforge/labels/maybe-done","name":"maybe-done","color":"bfe5bf"},{"url":"https://api.github.com/repos/fontforge/fontforge/labels/Windows","name":"Windows","color":"207de5"}],"state":"open","locked":false,"assignee":{"login":"jtanx","id":5137410,"avatar_url":"https://avatars.githubusercontent.com/u/5137410?v=3","gravatar_id":"","url":"https://api.github.com/users/jtanx","html_url":"https://github.com/jtanx","followers_url":"https://api.github.com/users/jtanx/followers","following_url":"https://api.github.com/users/jtanx/following{/other_user}","gists_url":"https://api.github.com/users/jtanx/gists{/gist_id}","starred_url":"https://api.github.com/users/jtanx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jtanx/subscriptions","organizations_url":"https://api.github.com/users/jtanx/orgs","repos_url":"https://api.github.com/users/jtanx/repos","events_url":"https://api.github.com/users/jtanx/events{/privacy}","received_events_url":"https://api.github.com/users/jtanx/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/fontforge/fontforge/milestones/13","html_url":"https://github.com/fontforge/fontforge/milestones/2015-03%20Release","labels_url":"https://api.github.com/repos/fontforge/fontforge/milestones/13/labels","id":998180,"number":13,"title":"2015-03 Release","description":"","creator":{"login":"davelab6","id":261579,"avatar_url":"https://avatars.githubusercontent.com/u/261579?v=3","gravatar_id":"","url":"https://api.github.com/users/davelab6","html_url":"https://github.com/davelab6","followers_url":"https://api.github.com/users/davelab6/followers","following_url":"https://api.github.com/users/davelab6/following{/other_user}","gists_url":"https://api.github.com/users/davelab6/gists{/gist_id}","starred_url":"https://api.github.com/users/davelab6/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/davelab6/subscriptions","organizations_url":"https://api.github.com/users/davelab6/orgs","repos_url":"https://api.github.com/users/davelab6/repos","events_url":"https://api.github.com/users/davelab6/events{/privacy}","received_events_url":"https://api.github.com/users/davelab6/received_events","type":"User","site_admin":false},"open_issues":10,"closed_issues":0,"state":"open","created_at":"2015-02-27T02:01:40Z","updated_at":"2015-02-27T15:52:11Z","due_on":"2015-03-27T04:00:00Z","closed_at":null},"comments":6,"created_at":"2014-10-16T00:43:40Z","updated_at":"2015-02-27T23:48:59Z","closed_at":null,"body":"User can easily crash the latest Windows FF by using Windows file path delimiters.  Other combinations give wildly different results.  Please pardon the number of examples, but I was looking for a pattern without looking at the code:\r\n\r\nStarting with initial \"Open Font\" dialog box, shows \"C:\\Users\\Tom\\Documents\" as list of 4 items:\r\n>        \"Documents\"\r\n>        \"Tom\"\r\n>        \"Users\"\r\n>        \"C:\"\r\n\r\n- Enter \"C:\\Toms\\\"  - crashes\r\n\r\n- Enter \"C:\\Toms\"   - crashes\r\n\r\n- Enter \"\\Toms\"     - \"Couldn't open font\" \"The requested file, Toms, does not exist\"\r\n\r\n- Enter \"\\Toms\\\"    - \"Couldn't open font\" \"The requested file, Toms, does not exist\"\r\n\r\n- Select \"C:\" from top list button\r\nEnter \"\\Toms\"     - bounces me back to original \"Documents\" location \r\n\r\n- Select \"C:\" from top list button\r\nEnter \"\\Toms\\\"    - bounces me back to original \"Documents\" location \r\n\r\n\r\n- Enter \"C:/Toms/\"  - works to display C:\\Toms\\\r\n\r\n- Enter \"C:/Toms\"   - \"Couldn't open font\" \"Toms is not in a known format ...\"\r\n\r\n- Enter \"//Toms/    - in dir list area \" Not Found\" \"No such file or directory\"\r\n\r\n- Enter \"//Toms     - \"Couldn't open font\" \"The requested file, Toms, does not exist\"\r\n\r\n- Enter \"C://\"      - Shows \"C://\",  in dir list area \" Not Implemented\" \"No support for browsing : C\"\r\n\r\n- Enter \"C:/\"       - works to display \"C:\\\" root directory\r\n\r\n- Select \"C:\" from top list button\r\nEnter \"//Toms/    - in dir list area \" Not Implemented\" \"No support for browsing : C\"\r\n\r\n- Select \"C:\" from top list button\r\nEnter \"//Toms\"    - bounces me back to original \"Documents\" location \r\n\r\n- Select \"C:\" from top list button\r\nEnter \"/Toms\"     - bounces me back to original \"Documents\" location \r\n\r\n- Select \"C:\" from top list button\r\nEnter \"/Toms/\"    - Shows \"C://Toms/\", in dir list area \" Not Implemented\" \"No support for browsing : C\"\r\n\r\n- Select \"C:\" from top list button\r\nEnter \"Toms\"      - works to display C:\\Toms\\\r\n\r\n\r\nStarted on command to see version:\r\n```\r\nCopyright (c) 2000-2014 by George Williams. See AUTHORS for Contributors.\r\n License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>\r\n with many parts BSD <http://fontforge.org/license.html>. Please read LICENSE.\r\n Based on sources from 11:53 AWST 14-Oct-2014-ML-TtfDb-D.\r\n Based on source from git with hash:f6ffb25d7732adcc8d32dae75bd90bda90250304\r\nno xdefs_filename!\r\nTESTING: getPixmapDir:C:/Program Files (x86)/FontForgeBuilds/share/fontforge/pixmaps\r\nTESTING: getShareDir:C:/Program Files (x86)/FontForgeBuilds/share/fontforge\r\nTESTING: GResourceProgramDir:C:/Program Files (x86)/FontForgeBuilds/bin\r\ntrying default theme:C:/Program Files (x86)/FontForgeBuilds/share/fontforge/pixmaps/resources\r\nfontforge 11:53 AWST 14-Oct-2014\r\nlibfontforge 20141014\r\n```\r\n\r\nBTW: tooltips popup with blank areas - no text\r\n\r\nBTW:  #1743 was forgotten"},"comment":{"url":"https://api.github.com/repos/fontforge/fontforge/issues/comments/76494372","html_url":"https://github.com/fontforge/fontforge/issues/1803#issuecomment-76494372","issue_url":"https://api.github.com/repos/fontforge/fontforge/issues/1803","id":76494372,"user":{"login":"jtanx","id":5137410,"avatar_url":"https://avatars.githubusercontent.com/u/5137410?v=3","gravatar_id":"","url":"https://api.github.com/users/jtanx","html_url":"https://github.com/jtanx","followers_url":"https://api.github.com/users/jtanx/followers","following_url":"https://api.github.com/users/jtanx/following{/other_user}","gists_url":"https://api.github.com/users/jtanx/gists{/gist_id}","starred_url":"https://api.github.com/users/jtanx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jtanx/subscriptions","organizations_url":"https://api.github.com/users/jtanx/orgs","repos_url":"https://api.github.com/users/jtanx/repos","events_url":"https://api.github.com/users/jtanx/events{/privacy}","received_events_url":"https://api.github.com/users/jtanx/received_events","type":"User","site_admin":false},"created_at":"2015-02-27T23:48:59Z","updated_at":"2015-02-27T23:48:59Z","body":"Since folders can also be treated as 'files' (e.g. UFO / sfdir), and without implementing more complex strategies of checking each folder to see if it's a UFO/sfdir, it's about as best as it can be.\r\n\r\ni.e. if a user enters a folder address without a trailing slash, it's treated as a potential UFO/sfdir. So the dialogue box closes and FF attempts to read it as such. When it fails because it's not a font, it reopens the file chooser in that location instead. This closing/opening of the file chooser is not ideal, but it works.\r\n\r\nAnother potential improvement could be to show all the drives available in the dropdown list, but you'd have to think about how this would work (since each drive is the 'root' directory but the dropdown list only allows for one 'root' directory as per Unix standards)\r\n"}}