{"action":"closed","issue":{"url":"https://api.github.com/repos/rasmusjp/umbraco-multi-url-picker/issues/75","repository_url":"https://api.github.com/repos/rasmusjp/umbraco-multi-url-picker","labels_url":"https://api.github.com/repos/rasmusjp/umbraco-multi-url-picker/issues/75/labels{/name}","comments_url":"https://api.github.com/repos/rasmusjp/umbraco-multi-url-picker/issues/75/comments","events_url":"https://api.github.com/repos/rasmusjp/umbraco-multi-url-picker/issues/75/events","html_url":"https://github.com/rasmusjp/umbraco-multi-url-picker/issues/75","id":311265268,"node_id":"MDU6SXNzdWUzMTEyNjUyNjg=","number":75,"title":"Toggle query string and other options","user":{"login":"ronaldbarendse","id":1051287,"node_id":"MDQ6VXNlcjEwNTEyODc=","avatar_url":"https://avatars2.githubusercontent.com/u/1051287?v=4","gravatar_id":"","url":"https://api.github.com/users/ronaldbarendse","html_url":"https://github.com/ronaldbarendse","followers_url":"https://api.github.com/users/ronaldbarendse/followers","following_url":"https://api.github.com/users/ronaldbarendse/following{/other_user}","gists_url":"https://api.github.com/users/ronaldbarendse/gists{/gist_id}","starred_url":"https://api.github.com/users/ronaldbarendse/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ronaldbarendse/subscriptions","organizations_url":"https://api.github.com/users/ronaldbarendse/orgs","repos_url":"https://api.github.com/users/ronaldbarendse/repos","events_url":"https://api.github.com/users/ronaldbarendse/events{/privacy}","received_events_url":"https://api.github.com/users/ronaldbarendse/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2018-04-04T14:54:39Z","updated_at":"2018-08-21T16:55:58Z","closed_at":"2018-08-21T16:55:58Z","author_association":"CONTRIBUTOR","body":"The query string is probably not needed by most editors and thus it should be possible to disable this field per data type (as pre value).\r\n\r\nThe query string field also allows adding a fragment/hash, so maybe this should be made clear by renaming it to 'Query string or hash' or just 'URL postfix'. Splitting this into 2 fields is probably overkill IMHO.\r\n\r\nCurrently, the overlay is based on the default Umbraco [linkpicker](https://github.com/umbraco/Umbraco-CMS/blob/dev-v7/src/Umbraco.Web.UI.Client/src/views/common/overlays/linkpicker/linkpicker.html), so adding an option for `showTarget` should also be fairly simple.\r\n\r\nAlso (and this is something that should probably be changed in Umbraco itself) the order of fields could be improved, because selecting content or media overwrites any URL and/or name that is entered before that... As suggested by @greystate on the Umbraco Community Slack, the overlay could be reworked to look like this (personally, I would add the target as last):\r\n\r\n![Linkpicker overlay mock-up](https://user-images.githubusercontent.com/1051287/38311855-75108e72-3820-11e8-829f-53d2476936ee.png)\r\n\r\nThe additional fields, like query string and target, could also be added as advanced options (like the redirect overlay of [Skybrud.Umbraco.Redirects](https://github.com/skybrud/Skybrud.Umbraco.Redirects); maybe even collapsed by default):\r\n\r\n![redirects](https://user-images.githubusercontent.com/1051287/38311569-cb81fba2-381f-11e8-909d-1fc3a27001da.png)"}}
{"actor":{"display_login":"rasmusjp"}}