payload: {"action":"created","comment":{"url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/pulls/comments/342123562","pull_request_review_id":311167206,"id":342123562,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDM0MjEyMzU2Mg==","diff_hunk":"@@ -655,6 +655,8 @@ tr.odd {\n   border-radius: 5px;\n   font-size:0.8em;\n   box-shadow: 2px 1px 4px #88888B;\n+  width: 80px;\n+  text-align: center;","path":"interface/themes/style_tan.css","position":5,"original_position":5,"commit_id":"e2506cca665bf61842bf52d4debb8a97d69a8967","original_commit_id":"e2506cca665bf61842bf52d4debb8a97d69a8967","user":{"login":"Ngai-E","id":22508583,"node_id":"MDQ6VXNlcjIyNTA4NTgz","avatar_url":"https://avatars1.githubusercontent.com/u/22508583?v=4","gravatar_id":"","url":"https://api.github.com/users/Ngai-E","html_url":"https://github.com/Ngai-E","followers_url":"https://api.github.com/users/Ngai-E/followers","following_url":"https://api.github.com/users/Ngai-E/following{/other_user}","gists_url":"https://api.github.com/users/Ngai-E/gists{/gist_id}","starred_url":"https://api.github.com/users/Ngai-E/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ngai-E/subscriptions","organizations_url":"https://api.github.com/users/Ngai-E/orgs","repos_url":"https://api.github.com/users/Ngai-E/repos","events_url":"https://api.github.com/users/Ngai-E/events{/privacy}","received_events_url":"https://api.github.com/users/Ngai-E/received_events","type":"User","site_admin":false},"body":"Try to narrow down your fix as much as possible to the affected file to avoid breaking styles or code in other modules, modifying this theme file like this might cause  a global effect in codebase. create a style class for that particular use case a apply the style other than applying is to all button input types. Please try not to do it here except you are going to create another class name.","created_at":"2019-11-04T15:52:56Z","updated_at":"2019-11-04T15:53:06Z","html_url":"https://github.com/LibreHealthIO/lh-ehr/pull/1538#discussion_r342123562","pull_request_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/pulls/1538","author_association":"COLLABORATOR","_links":{"self":{"href":"https://api.github.com/repos/LibreHealthIO/lh-ehr/pulls/comments/342123562"},"html":{"href":"https://github.com/LibreHealthIO/lh-ehr/pull/1538#discussion_r342123562"},"pull_request":{"href":"https://api.github.com/repos/LibreHealthIO/lh-ehr/pulls/1538"}}},"pull_request":{"url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/pulls/1538","id":335921989,"node_id":"MDExOlB1bGxSZXF1ZXN0MzM1OTIxOTg5","html_url":"https://github.com/LibreHealthIO/lh-ehr/pull/1538","diff_url":"https://github.com/LibreHealthIO/lh-ehr/pull/1538.diff","patch_url":"https://github.com/LibreHealthIO/lh-ehr/pull/1538.patch","issue_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/issues/1538","number":1538,"state":"open","locked":false,"title":"Fixes Arrow icons overlapped by text on ‘Previous’ and ‘Next’ Buttons on Patient Finder","user":{"login":"j-pro","id":1905329,"node_id":"MDQ6VXNlcjE5MDUzMjk=","avatar_url":"https://avatars2.githubusercontent.com/u/1905329?v=4","gravatar_id":"","url":"https://api.github.com/users/j-pro","html_url":"https://github.com/j-pro","followers_url":"https://api.github.com/users/j-pro/followers","following_url":"https://api.github.com/users/j-pro/following{/other_user}","gists_url":"https://api.github.com/users/j-pro/gists{/gist_id}","starred_url":"https://api.github.com/users/j-pro/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/j-pro/subscriptions","organizations_url":"https://api.github.com/users/j-pro/orgs","repos_url":"https://api.github.com/users/j-pro/repos","events_url":"https://api.github.com/users/j-pro/events{/privacy}","received_events_url":"https://api.github.com/users/j-pro/received_events","type":"User","site_admin":false},"body":"this commit fixes #1412\r\n\r\nA) Outreachy Username: j-pro \r\nB) Issue title: Arrow icons overlapped by text on ‘Previous’ and ‘Next’ Buttons on Patient Finder  \r\nC) Report Date: Oct 26, 2019  \r\nD) Sites Affected: Documentation Site \r\nE) OS/ browser used: Windows / Chrome, Firefox \r\nF) EHR Workflow module: Patient Finder \r\nG) Steps to Reproduce the Issue:  \r\n\r\n1. Login with your account credentials. \r\n2. In the top navigation, take your cursor to ‘Patient/Client’. \r\n3. A dropdown menu will appear.  Click on ‘Finder’. \r\n4. ‘Patient Finder’ will open.  Scroll till the end where you find ‘Previous’ and ‘Next’ Buttons. \r\n\r\nH) Expected Behavior: There should be some space between the icon and text of the Buttons, so that both the text and icon can be clearly seen.  \r\n\r\nI) What Actually Happened: The icon and text are overlapped in both the ‘Next’ and ‘Previous’ Buttons, causing some difficulties in reading the text. \r\n\r\nJ) Screenshots with Explanation:  \r\n\r\n ![npb-01](https://user-images.githubusercontent.com/1905329/68075976-214cfb80-fdd7-11e9-91b5-6b7af106a39a.png)\r\nThe ‘Next’ and ‘Previous’ Buttons have text and icon overlapped. \r\n \r\n\r\n![npb-02](https://user-images.githubusercontent.com/1905329/68075978-214cfb80-fdd7-11e9-9128-1248125cb009.png)\r\nThe highlighted code represents the ‘Next’ Button.  \r\n\r\nK) Estimation of Severity: \r\nMinor:  It is somehow distracting or confusing but workflow can continue correctly.  \r\n\r\nL) Solution:  \r\n \r\nI think the width of the Buttons is not defined, so they are taking only the width of text, excluding the arrow icons.  \r\n \r\n![npb-03](https://user-images.githubusercontent.com/1905329/68075979-214cfb80-fdd7-11e9-9251-aa7cb4861184.png) \r\nI tried adding `width: 80px`.  The ‘Next’ text appeared in the right side and still got overlapped to the icon.  So I center aligned the text as shown in screenshot, and it seems to be the solution for now. \r\n \r\n\r\n\r\n\r\n\r\n\r\n","created_at":"2019-11-02T19:32:20Z","updated_at":"2019-11-04T15:53:06Z","closed_at":null,"merged_at":null,"merge_commit_sha":"68864fa04192108e21fc86d65ce826727c616fab","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/pulls/1538/commits","review_comments_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/pulls/1538/comments","review_comment_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/pulls/comments{/number}","comments_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/issues/1538/comments","statuses_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/statuses/e2506cca665bf61842bf52d4debb8a97d69a8967","head":{"label":"j-pro:fix-icon-overlap-in-next-prev","ref":"fix-icon-overlap-in-next-prev","sha":"e2506cca665bf61842bf52d4debb8a97d69a8967","user":{"login":"j-pro","id":1905329,"node_id":"MDQ6VXNlcjE5MDUzMjk=","avatar_url":"https://avatars2.githubusercontent.com/u/1905329?v=4","gravatar_id":"","url":"https://api.github.com/users/j-pro","html_url":"https://github.com/j-pro","followers_url":"https://api.github.com/users/j-pro/followers","following_url":"https://api.github.com/users/j-pro/following{/other_user}","gists_url":"https://api.github.com/users/j-pro/gists{/gist_id}","starred_url":"https://api.github.com/users/j-pro/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/j-pro/subscriptions","organizations_url":"https://api.github.com/users/j-pro/orgs","repos_url":"https://api.github.com/users/j-pro/repos","events_url":"https://api.github.com/users/j-pro/events{/privacy}","received_events_url":"https://api.github.com/users/j-pro/received_events","type":"User","site_admin":false},"repo":{"id":217895779,"node_id":"MDEwOlJlcG9zaXRvcnkyMTc4OTU3Nzk=","name":"lh-ehr","full_name":"j-pro/lh-ehr","private":false,"owner":{"login":"j-pro","id":1905329,"node_id":"MDQ6VXNlcjE5MDUzMjk=","avatar_url":"https://avatars2.githubusercontent.com/u/1905329?v=4","gravatar_id":"","url":"https://api.github.com/users/j-pro","html_url":"https://github.com/j-pro","followers_url":"https://api.github.com/users/j-pro/followers","following_url":"https://api.github.com/users/j-pro/following{/other_user}","gists_url":"https://api.github.com/users/j-pro/gists{/gist_id}","starred_url":"https://api.github.com/users/j-pro/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/j-pro/subscriptions","organizations_url":"https://api.github.com/users/j-pro/orgs","repos_url":"https://api.github.com/users/j-pro/repos","events_url":"https://api.github.com/users/j-pro/events{/privacy}","received_events_url":"https://api.github.com/users/j-pro/received_events","type":"User","site_admin":false},"html_url":"https://github.com/j-pro/lh-ehr","description":"LibreHealth EHR - Free Open Source Electronic Health Records","fork":true,"url":"https://api.github.com/repos/j-pro/lh-ehr","forks_url":"https://api.github.com/repos/j-pro/lh-ehr/forks","keys_url":"https://api.github.com/repos/j-pro/lh-ehr/keys{/key_id}","collaborators_url":"https://api.github.com/repos/j-pro/lh-ehr/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/j-pro/lh-ehr/teams","hooks_url":"https://api.github.com/repos/j-pro/lh-ehr/hooks","issue_events_url":"https://api.github.com/repos/j-pro/lh-ehr/issues/events{/number}","events_url":"https://api.github.com/repos/j-pro/lh-ehr/events","assignees_url":"https://api.github.com/repos/j-pro/lh-ehr/assignees{/user}","branches_url":"https://api.github.com/repos/j-pro/lh-ehr/branches{/branch}","tags_url":"https://api.github.com/repos/j-pro/lh-ehr/tags","blobs_url":"https://api.github.com/repos/j-pro/lh-ehr/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/j-pro/lh-ehr/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/j-pro/lh-ehr/git/refs{/sha}","trees_url":"https://api.github.com/repos/j-pro/lh-ehr/git/trees{/sha}","statuses_url":"https://api.github.com/repos/j-pro/lh-ehr/statuses/{sha}","languages_url":"https://api.github.com/repos/j-pro/lh-ehr/languages","stargazers_url":"https://api.github.com/repos/j-pro/lh-ehr/stargazers","contributors_url":"https://api.github.com/repos/j-pro/lh-ehr/contributors","subscribers_url":"https://api.github.com/repos/j-pro/lh-ehr/subscribers","subscription_url":"https://api.github.com/repos/j-pro/lh-ehr/subscription","commits_url":"https://api.github.com/repos/j-pro/lh-ehr/commits{/sha}","git_commits_url":"https://api.github.com/repos/j-pro/lh-ehr/git/commits{/sha}","comments_url":"https://api.github.com/repos/j-pro/lh-ehr/comments{/number}","issue_comment_url":"https://api.github.com/repos/j-pro/lh-ehr/issues/comments{/number}","contents_url":"https://api.github.com/repos/j-pro/lh-ehr/contents/{+path}","compare_url":"https://api.github.com/repos/j-pro/lh-ehr/compare/{base}...{head}","merges_url":"https://api.github.com/repos/j-pro/lh-ehr/merges","archive_url":"https://api.github.com/repos/j-pro/lh-ehr/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/j-pro/lh-ehr/downloads","issues_url":"https://api.github.com/repos/j-pro/lh-ehr/issues{/number}","pulls_url":"https://api.github.com/repos/j-pro/lh-ehr/pulls{/number}","milestones_url":"https://api.github.com/repos/j-pro/lh-ehr/milestones{/number}","notifications_url":"https://api.github.com/repos/j-pro/lh-ehr/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/j-pro/lh-ehr/labels{/name}","releases_url":"https://api.github.com/repos/j-pro/lh-ehr/releases{/id}","deployments_url":"https://api.github.com/repos/j-pro/lh-ehr/deployments","created_at":"2019-10-27T18:10:24Z","updated_at":"2019-10-27T18:10:25Z","pushed_at":"2019-11-02T19:24:54Z","git_url":"git://github.com/j-pro/lh-ehr.git","ssh_url":"git@github.com:j-pro/lh-ehr.git","clone_url":"https://github.com/j-pro/lh-ehr.git","svn_url":"https://github.com/j-pro/lh-ehr","homepage":"","size":454220,"stargazers_count":0,"watchers_count":0,"language":null,"has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"disabled":false,"open_issues_count":0,"license":{"key":"other","name":"Other","spdx_id":"NOASSERTION","url":null,"node_id":"MDc6TGljZW5zZTA="},"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"LibreHealthIO:master","ref":"master","sha":"3e64806dd283a556602fa7f30631e28f04d698fd","user":{"login":"LibreHealthIO","id":19714529,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE5NzE0NTI5","avatar_url":"https://avatars3.githubusercontent.com/u/19714529?v=4","gravatar_id":"","url":"https://api.github.com/users/LibreHealthIO","html_url":"https://github.com/LibreHealthIO","followers_url":"https://api.github.com/users/LibreHealthIO/followers","following_url":"https://api.github.com/users/LibreHealthIO/following{/other_user}","gists_url":"https://api.github.com/users/LibreHealthIO/gists{/gist_id}","starred_url":"https://api.github.com/users/LibreHealthIO/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LibreHealthIO/subscriptions","organizations_url":"https://api.github.com/users/LibreHealthIO/orgs","repos_url":"https://api.github.com/users/LibreHealthIO/repos","events_url":"https://api.github.com/users/LibreHealthIO/events{/privacy}","received_events_url":"https://api.github.com/users/LibreHealthIO/received_events","type":"Organization","site_admin":false},"repo":{"id":63452821,"node_id":"MDEwOlJlcG9zaXRvcnk2MzQ1MjgyMQ==","name":"lh-ehr","full_name":"LibreHealthIO/lh-ehr","private":false,"owner":{"login":"LibreHealthIO","id":19714529,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE5NzE0NTI5","avatar_url":"https://avatars3.githubusercontent.com/u/19714529?v=4","gravatar_id":"","url":"https://api.github.com/users/LibreHealthIO","html_url":"https://github.com/LibreHealthIO","followers_url":"https://api.github.com/users/LibreHealthIO/followers","following_url":"https://api.github.com/users/LibreHealthIO/following{/other_user}","gists_url":"https://api.github.com/users/LibreHealthIO/gists{/gist_id}","starred_url":"https://api.github.com/users/LibreHealthIO/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LibreHealthIO/subscriptions","organizations_url":"https://api.github.com/users/LibreHealthIO/orgs","repos_url":"https://api.github.com/users/LibreHealthIO/repos","events_url":"https://api.github.com/users/LibreHealthIO/events{/privacy}","received_events_url":"https://api.github.com/users/LibreHealthIO/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/LibreHealthIO/lh-ehr","description":"LibreHealth EHR - Free Open Source Electronic Health Records","fork":false,"url":"https://api.github.com/repos/LibreHealthIO/lh-ehr","forks_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/forks","keys_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/keys{/key_id}","collaborators_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/teams","hooks_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/hooks","issue_events_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/issues/events{/number}","events_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/events","assignees_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/assignees{/user}","branches_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/branches{/branch}","tags_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/tags","blobs_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/git/refs{/sha}","trees_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/git/trees{/sha}","statuses_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/statuses/{sha}","languages_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/languages","stargazers_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/stargazers","contributors_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/contributors","subscribers_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/subscribers","subscription_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/subscription","commits_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/commits{/sha}","git_commits_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/git/commits{/sha}","comments_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/comments{/number}","issue_comment_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/issues/comments{/number}","contents_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/contents/{+path}","compare_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/compare/{base}...{head}","merges_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/merges","archive_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/downloads","issues_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/issues{/number}","pulls_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/pulls{/number}","milestones_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/milestones{/number}","notifications_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/labels{/name}","releases_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/releases{/id}","deployments_url":"https://api.github.com/repos/LibreHealthIO/lh-ehr/deployments","created_at":"2016-07-15T21:50:55Z","updated_at":"2019-10-30T07:38:19Z","pushed_at":"2019-11-04T11:37:42Z","git_url":"git://github.com/LibreHealthIO/lh-ehr.git","ssh_url":"git@github.com:LibreHealthIO/lh-ehr.git","clone_url":"https://github.com/LibreHealthIO/lh-ehr.git","svn_url":"https://github.com/LibreHealthIO/lh-ehr","homepage":"","size":454298,"stargazers_count":105,"watchers_count":105,"language":"PHP","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":162,"mirror_url":null,"archived":false,"disabled":false,"open_issues_count":177,"license":{"key":"other","name":"Other","spdx_id":"NOASSERTION","url":null,"node_id":"MDc6TGljZW5zZTA="},"forks":162,"open_issues":177,"watchers":105,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/LibreHealthIO/lh-ehr/pulls/1538"},"html":{"href":"https://github.com/LibreHealthIO/lh-ehr/pull/1538"},"issue":{"href":"https://api.github.com/repos/LibreHealthIO/lh-ehr/issues/1538"},"comments":{"href":"https://api.github.com/repos/LibreHealthIO/lh-ehr/issues/1538/comments"},"review_comments":{"href":"https://api.github.com/repos/LibreHealthIO/lh-ehr/pulls/1538/comments"},"review_comment":{"href":"https://api.github.com/repos/LibreHealthIO/lh-ehr/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/LibreHealthIO/lh-ehr/pulls/1538/commits"},"statuses":{"href":"https://api.github.com/repos/LibreHealthIO/lh-ehr/statuses/e2506cca665bf61842bf52d4debb8a97d69a8967"}},"author_association":"NONE"}}
other: {"actor":{"display_login":"Ngai-E"}}