)]}'
{"src/js/components/deployment_plan/DeploymentPlanStep.js":[{"author":{"_account_id":7509,"name":"Jiri Tomasek","email":"jtomasek@redhat.com","username":"jtomasek"},"change_message_id":"9862148f22fc7c1e7df264953bccd8875b89f442","unresolved":false,"context_lines":[{"line_number":6,"context_line":"      \u003ch3\u003e"},{"line_number":7,"context_line":"        \u003cspan\u003e{title}\u003c/span\u003e"},{"line_number":8,"context_line":"        {tooltip ?"},{"line_number":9,"context_line":"          \u003cspan is data-tooltip\u003d{tooltip} class\u003d\"tooltip-right\"\u003e"},{"line_number":10,"context_line":"            \u003cspan className\u003d\"pficon pficon-info\"/\u003e"},{"line_number":11,"context_line":"          \u003c/span\u003e"},{"line_number":12,"context_line":"          : null"}],"source_content_type":"text/javascript","patch_set":4,"id":"da36d5c6_e3288935","line":9,"range":{"start_line":9,"start_character":16,"end_line":9,"end_character":18},"updated":"2017-02-24 15:22:37.000000000","message":"this attribute is not used it seems","commit_id":"e5fb8b6c35435b3c302d9f1240d5fa14062cf23b"},{"author":{"_account_id":7509,"name":"Jiri Tomasek","email":"jtomasek@redhat.com","username":"jtomasek"},"change_message_id":"9862148f22fc7c1e7df264953bccd8875b89f442","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    \u003cli className\u003d{disabled ? \u0027disabled\u0027 : null}\u003e"},{"line_number":6,"context_line":"      \u003ch3\u003e"},{"line_number":7,"context_line":"        \u003cspan\u003e{title}\u003c/span\u003e"},{"line_number":8,"context_line":"        {tooltip ?"},{"line_number":9,"context_line":"          \u003cspan is data-tooltip\u003d{tooltip} class\u003d\"tooltip-right\"\u003e"},{"line_number":10,"context_line":"            \u003cspan className\u003d\"pficon pficon-info\"/\u003e"},{"line_number":11,"context_line":"          \u003c/span\u003e"},{"line_number":12,"context_line":"          : null"},{"line_number":13,"context_line":"        }"},{"line_number":14,"context_line":"      \u003c/h3\u003e"},{"line_number":15,"context_line":"      {children}"},{"line_number":16,"context_line":"    \u003c/li\u003e"}],"source_content_type":"text/javascript","patch_set":4,"id":"da36d5c6_838b4d1e","line":13,"range":{"start_line":8,"start_character":8,"end_line":13,"end_character":9},"updated":"2017-02-24 15:22:37.000000000","message":"This whole part should be extracted into a separate Tooltip component which will render if it\u0027s tooltip prop is not empty.\nThe props for the component should be: content, position and children. Position can have a default. You can use onMouseOver and onMouseOut to toggle the tooltip.\n\nResulting usage is:\n\u003cTooltip content\u003d\"The text of the tooltip\"\n         position\u003d\"right\"\u003e\n  \u003cspan className\u003d\"pficon pficon-info\"/\u003e\n\u003c/Tooltip\u003e","commit_id":"e5fb8b6c35435b3c302d9f1240d5fa14062cf23b"},{"author":{"_account_id":7509,"name":"Jiri Tomasek","email":"jtomasek@redhat.com","username":"jtomasek"},"change_message_id":"9862148f22fc7c1e7df264953bccd8875b89f442","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"DeploymentPlanStep.defaultProps \u003d {"},{"line_number":28,"context_line":"  disabled: false,"},{"line_number":29,"context_line":"  tooltip: \u0027\u0027"},{"line_number":30,"context_line":"};"}],"source_content_type":"text/javascript","patch_set":4,"id":"da36d5c6_c381c5fc","line":29,"updated":"2017-02-24 15:22:37.000000000","message":"not needed as it is not rendered when it is empty","commit_id":"e5fb8b6c35435b3c302d9f1240d5fa14062cf23b"},{"author":{"_account_id":7509,"name":"Jiri Tomasek","email":"jtomasek@redhat.com","username":"jtomasek"},"change_message_id":"7df1d6dd8ff4c05294fdc345f5504205196e98f5","unresolved":false,"context_lines":[{"line_number":6,"context_line":"      \u003ch3\u003e"},{"line_number":7,"context_line":"        \u003cspan\u003e{title}\u003c/span\u003e"},{"line_number":8,"context_line":"        {tooltip ?"},{"line_number":9,"context_line":"          \u003cspan is data-tooltip\u003d{tooltip} class\u003d\"tooltip-right\"\u003e"},{"line_number":10,"context_line":"            \u003cspan className\u003d\"pficon pficon-info\"/\u003e"},{"line_number":11,"context_line":"          \u003c/span\u003e"},{"line_number":12,"context_line":"          : null"}],"source_content_type":"text/javascript","patch_set":5,"id":"da36d5c6_175cb7ec","line":9,"range":{"start_line":9,"start_character":16,"end_line":9,"end_character":19},"updated":"2017-02-27 18:48:02.000000000","message":"not used","commit_id":"6faf3d588fcde435c9c8f2ac6b23d53fcdeb2378"},{"author":{"_account_id":10112,"name":"Jason E. Rist","email":"jrist@redhat.com","username":"jrist"},"change_message_id":"9437722f065ae26bfd09945d18f74adc95b5eab7","unresolved":false,"context_lines":[{"line_number":6,"context_line":"      \u003ch3\u003e"},{"line_number":7,"context_line":"        \u003cspan\u003e{title}\u003c/span\u003e"},{"line_number":8,"context_line":"        {tooltip ?"},{"line_number":9,"context_line":"          \u003cspan is data-tooltip\u003d{tooltip} class\u003d\"tooltip-right\"\u003e"},{"line_number":10,"context_line":"            \u003cspan className\u003d\"pficon pficon-info\"/\u003e"},{"line_number":11,"context_line":"          \u003c/span\u003e"},{"line_number":12,"context_line":"          : null"}],"source_content_type":"text/javascript","patch_set":5,"id":"da36d5c6_1d5775cb","line":9,"range":{"start_line":9,"start_character":16,"end_line":9,"end_character":19},"in_reply_to":"da36d5c6_175cb7ec","updated":"2017-02-27 20:22:49.000000000","message":"Yes, this is an undocumented feature of React that lets us use custom attributes like data-tooltip.","commit_id":"6faf3d588fcde435c9c8f2ac6b23d53fcdeb2378"},{"author":{"_account_id":7509,"name":"Jiri Tomasek","email":"jtomasek@redhat.com","username":"jtomasek"},"change_message_id":"ad63471a564b3b43b184b203bbf6d301d64415ef","unresolved":false,"context_lines":[{"line_number":6,"context_line":"      \u003ch3\u003e"},{"line_number":7,"context_line":"        \u003cspan\u003e{title}\u003c/span\u003e"},{"line_number":8,"context_line":"        {tooltip ?"},{"line_number":9,"context_line":"          \u003cspan is data-tooltip\u003d{tooltip} class\u003d\"tooltip-right\"\u003e"},{"line_number":10,"context_line":"            \u003cspan className\u003d\"pficon pficon-info\"/\u003e"},{"line_number":11,"context_line":"          \u003c/span\u003e"},{"line_number":12,"context_line":"          : null"}],"source_content_type":"text/javascript","patch_set":5,"id":"ba2be162_96823bff","line":9,"range":{"start_line":9,"start_character":16,"end_line":9,"end_character":19},"in_reply_to":"da36d5c6_1d5775cb","updated":"2017-03-01 15:32:24.000000000","message":"\u003cspan data-tooltip\u003d{tooltip} className\u003d\"tooltip-right\"\u003e\nworks without any problem for me...\nDocs mention that data-* attributes are supported (https://facebook.github.io/react/docs/dom-elements.html#all-supported-html-attributes)","commit_id":"6faf3d588fcde435c9c8f2ac6b23d53fcdeb2378"}],"src/less/ui/Tooltips.less":[{"author":{"_account_id":7509,"name":"Jiri Tomasek","email":"jtomasek@redhat.com","username":"jtomasek"},"change_message_id":"9862148f22fc7c1e7df264953bccd8875b89f442","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"da36d5c6_3e71d876","updated":"2017-02-24 15:22:37.000000000","message":"This seems to reimplement whole tooltip behavior and styling instead of using what patternfly/bootstrap does. See http://getbootstrap.com/javascript/#markup-1 for a generated markup which their code produces. I think it would be better to do it that way rather than using :after pseudo element. This approach also allows us to turn any component or element into Tooltip trigger.","commit_id":"e5fb8b6c35435b3c302d9f1240d5fa14062cf23b"}]}