{"action":"created","issue":{"url":"https://api.github.com/repos/mozfr/transvision/issues/769","repository_url":"https://api.github.com/repos/mozfr/transvision","labels_url":"https://api.github.com/repos/mozfr/transvision/issues/769/labels{/name}","comments_url":"https://api.github.com/repos/mozfr/transvision/issues/769/comments","events_url":"https://api.github.com/repos/mozfr/transvision/issues/769/events","html_url":"https://github.com/mozfr/transvision/pull/769","id":164453644,"number":769,"title":"Transvision visual refresh","user":{"login":"TheoChevalier","id":1294206,"avatar_url":"https://avatars.githubusercontent.com/u/1294206?v=3","gravatar_id":"","url":"https://api.github.com/users/TheoChevalier","html_url":"https://github.com/TheoChevalier","followers_url":"https://api.github.com/users/TheoChevalier/followers","following_url":"https://api.github.com/users/TheoChevalier/following{/other_user}","gists_url":"https://api.github.com/users/TheoChevalier/gists{/gist_id}","starred_url":"https://api.github.com/users/TheoChevalier/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TheoChevalier/subscriptions","organizations_url":"https://api.github.com/users/TheoChevalier/orgs","repos_url":"https://api.github.com/users/TheoChevalier/repos","events_url":"https://api.github.com/users/TheoChevalier/events{/privacy}","received_events_url":"https://api.github.com/users/TheoChevalier/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-07-08T03:43:21Z","updated_at":"2016-07-08T11:45:02Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mozfr/transvision/pulls/769","html_url":"https://github.com/mozfr/transvision/pull/769","diff_url":"https://github.com/mozfr/transvision/pull/769.diff","patch_url":"https://github.com/mozfr/transvision/pull/769.patch"},"body":"I’ve been working on this last week, I think it’s in a good shape to start gathering feedback.\r\nYou can play with it here: https://dev.theochevalier.fr/\r\n\r\nIt’s a small visual update to refresh the look & feel until we rethink the UI (eventually).\r\nI don’t expect these changes to be disruptive for our users, but of course you may think differently. I could go further with the UI changes, but one step at a time.\r\n\r\nTo sum up the current changes:\r\n\r\n- Design a bit more flat for consistency with the logo (and we were almost there anyway)\r\n- Reducing the number of colors we’re using for consistency\r\n- Switching from Open Sans to Fira Sans, and generally reducing font-weight\r\n- Start moving away from our default \"everything is centered\" design (still thinking about what else can be done)\r\n- Block the menu button on the right for large screens (got feedback from delphine that it’s quite hard to notice)\r\n- Many other small details to make the UI less clumsy\r\n\r\nBefore I forget, commits 1 and 4 need to be removed before merging (webhook changes to keep the demo server up-to-date)\r\n\r\n@flodolo @pascalchevrel @SkySymbol and others, see any show-stopper? Like it/Don’t like it?\r\n\r\nNote: quite broken with IE but I don’t think we should care (and works well with Edge anyway)"},"comment":{"url":"https://api.github.com/repos/mozfr/transvision/issues/comments/231339869","html_url":"https://github.com/mozfr/transvision/pull/769#issuecomment-231339869","issue_url":"https://api.github.com/repos/mozfr/transvision/issues/769","id":231339869,"user":{"login":"TheoChevalier","id":1294206,"avatar_url":"https://avatars.githubusercontent.com/u/1294206?v=3","gravatar_id":"","url":"https://api.github.com/users/TheoChevalier","html_url":"https://github.com/TheoChevalier","followers_url":"https://api.github.com/users/TheoChevalier/followers","following_url":"https://api.github.com/users/TheoChevalier/following{/other_user}","gists_url":"https://api.github.com/users/TheoChevalier/gists{/gist_id}","starred_url":"https://api.github.com/users/TheoChevalier/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TheoChevalier/subscriptions","organizations_url":"https://api.github.com/users/TheoChevalier/orgs","repos_url":"https://api.github.com/users/TheoChevalier/repos","events_url":"https://api.github.com/users/TheoChevalier/events{/privacy}","received_events_url":"https://api.github.com/users/TheoChevalier/received_events","type":"User","site_admin":false},"created_at":"2016-07-08T11:45:02Z","updated_at":"2016-07-08T11:45:02Z","body":"Thanks for the feedback ;)\r\n\r\n1/ I agree with Fira Sans regular being too bold. I didn’t dare trying because I thought it would be too thin, but Fira Sans Light looks indeed quite nice with main content. Switching to it and dropping Fira Sans regular (page will be faster to load).\r\nAccording to Font Squirrel languages section, Fira Sans supports the same languages as Open Sans + Míkmaq, Sami (Northern), Ulithian, so I think we got covered. And this was a release from 2013, we’re using the latest.\r\n\r\nThe only issue I see is Romanian diacritics but I think it was already broken. For the same string sometimes they are displayed properly, sometimes not. IIRC, mozilla.org got that fixed with a specific version of Open Sans (or hacked it?)\r\nhttps://transvision.mozfr.org/?sourcelocale=en-US&locale=ro&repo=mozilla_org&search_type=entities&recherche=mozilla_org/firefox/accounts.lang:315b9ec0\r\nhttps://dev.theochevalier.fr/?sourcelocale=en-US&locale=ro&repo=mozilla_org&search_type=entities&recherche=mozilla_org/firefox/accounts.lang:315b9ec0\r\n\r\n2/ I also agree with that and will revert the change unless I can propose something with more left-aligned elements."}}
{"actor":{"display_login":"TheoChevalier"}}