{"action":"created","issue":{"url":"https://api.github.com/repos/open-craft/problem-builder/issues/29","labels_url":"https://api.github.com/repos/open-craft/problem-builder/issues/29/labels{/name}","comments_url":"https://api.github.com/repos/open-craft/problem-builder/issues/29/comments","events_url":"https://api.github.com/repos/open-craft/problem-builder/issues/29/events","html_url":"https://github.com/open-craft/problem-builder/pull/29","id":73348083,"number":29,"title":"Presentation updates, part 2","user":{"login":"e-kolpakov","id":3330048,"avatar_url":"https://avatars.githubusercontent.com/u/3330048?v=3","gravatar_id":"","url":"https://api.github.com/users/e-kolpakov","html_url":"https://github.com/e-kolpakov","followers_url":"https://api.github.com/users/e-kolpakov/followers","following_url":"https://api.github.com/users/e-kolpakov/following{/other_user}","gists_url":"https://api.github.com/users/e-kolpakov/gists{/gist_id}","starred_url":"https://api.github.com/users/e-kolpakov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/e-kolpakov/subscriptions","organizations_url":"https://api.github.com/users/e-kolpakov/orgs","repos_url":"https://api.github.com/users/e-kolpakov/repos","events_url":"https://api.github.com/users/e-kolpakov/events{/privacy}","received_events_url":"https://api.github.com/users/e-kolpakov/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-05-05T15:33:35Z","updated_at":"2015-05-06T14:45:28Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/open-craft/problem-builder/pulls/29","html_url":"https://github.com/open-craft/problem-builder/pull/29","diff_url":"https://github.com/open-craft/problem-builder/pull/29.diff","patch_url":"https://github.com/open-craft/problem-builder/pull/29.patch"},"body":"**Fixes and improvements:**\r\n* Give all the tables of the dashboard the same width (should be ~700px to give enough place to display long titles), and center the tables\r\n* Increase the width of the color codes in the table, to make them squarred instead of rectangular\r\n* Make the label for the \"Average\" row in the tables semi-bold (font-weight: 600)\r\n* Change aria-labels to off-screen text, as per Mark Sadeki's accessibility review: https://github.com/open-craft/problem-builder/pull/16#discussion_r29346587\r\n* In the export HTML:\r\n** Use left alignment for text within the tables, which is currently shown centered<\r\n** Apply the changes done to the table in the LMS: center the tables, use the same width for all tables (~700px - do not make it stretch to all available space, as widows can be big and it will look ugly), make the color codes squared, make the label for the &quot;Average&quot; row squarred\r\n* In Problem Builder, allow to customize the title for the Messages, which by default reads \"Feedback\"\r\n\r\n**Sandbox:** [LMS](http://sandbox3.opencraft.com/), [Studio](http://sandbox3.opencraft.com:18080)\r\n**JIRA Ticket:** [OSPR-575](https://openedx.atlassian.net/browse/OSPR-575)\r\n**Merge deadline:** preferably Friday, May 8 - this fixes are needed for course starting very soon.\r\n\r\n**Test instructions:**\r\n1. Set up a couple of mentoring XBlocks.\r\n2. Set up Dashboard block to draw data from those blocks\r\n3. Set custom value for \"Feedback Header\" for one of mentoring XBlocks, add completion message to this XBlock\r\n4. Publish everything\r\n5. Go to LMS, provide answers for mentoring blocks. Observe that overriding feedback messages header works - when mentoring with custom \"Feedback Header\" and \"Completion message\" is submitted, both configured header and message is shown (If \"Feedback\" is shown instead of overridden value - it is a bug).\r\n6. Navigate to dashboard block\r\n\r\nObserve the following:\r\n* All tables are centered and have 700px width\r\n* \"Average\" row caption is semibold (font-weight: 600)\r\n* Color-codes are much wider (30px) - should be square if question text fit to one line\r\n* Clicking \"Download Report\" (surprisingly) downloads a report. The report contains a copy of dashboard block visual representation and tables. All observations above applies to report content (table width, color codes and semi-bold \"Average\"). In addition, text in tables is left-aligned (it used to be centered)."},"comment":{"url":"https://api.github.com/repos/open-craft/problem-builder/issues/comments/99496960","html_url":"https://github.com/open-craft/problem-builder/pull/29#issuecomment-99496960","issue_url":"https://api.github.com/repos/open-craft/problem-builder/issues/29","id":99496960,"user":{"login":"sarina","id":1985317,"avatar_url":"https://avatars.githubusercontent.com/u/1985317?v=3","gravatar_id":"","url":"https://api.github.com/users/sarina","html_url":"https://github.com/sarina","followers_url":"https://api.github.com/users/sarina/followers","following_url":"https://api.github.com/users/sarina/following{/other_user}","gists_url":"https://api.github.com/users/sarina/gists{/gist_id}","starred_url":"https://api.github.com/users/sarina/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sarina/subscriptions","organizations_url":"https://api.github.com/users/sarina/orgs","repos_url":"https://api.github.com/users/sarina/repos","events_url":"https://api.github.com/users/sarina/events{/privacy}","received_events_url":"https://api.github.com/users/sarina/received_events","type":"User","site_admin":false},"created_at":"2015-05-06T14:45:28Z","updated_at":"2015-05-06T14:45:28Z","body":"@cptvitamin can you take a look at the a11y changes here? (you reviewed https://github.com/open-craft/problem-builder/pull/16 a few days ago, this is the response)"}}