D10209: Added setCursor() call in the CloseButton constructor

2018-01-31 Thread Nathaniel Graham
ngraham added a reviewer: Okular. REPOSITORY R223 Okular REVISION DETAIL https://phabricator.kde.org/D10209 To: dileepsankhla, #okular Cc: ngraham, #okular, michaelweghorn, aacid

D10209: Added setCursor() call in the CloseButton constructor

2018-01-31 Thread Nathaniel Graham
ngraham added a comment. Thanks for the patch! Can you indicate your testing, and perhaps provide a screenshot that shows this working? Also, the title of the revision is really better suited to being in the summary. For the title, try to briefly explain the overall purpose. Maybe somet

D10209: Added setCursor() call in the CloseButton constructor

2018-01-31 Thread Nathaniel Graham
ngraham edited the summary of this revision. REPOSITORY R223 Okular REVISION DETAIL https://phabricator.kde.org/D10209 To: dileepsankhla Cc: #okular, michaelweghorn, ngraham, aacid

D10209: Added setCursor() call in the CloseButton constructor

2018-01-31 Thread Dileep Sankhla
dileepsankhla created this revision. Restricted Application added a subscriber: Okular. Restricted Application added a project: Okular. dileepsankhla requested review of this revision. REPOSITORY R223 Okular BRANCH master REVISION DETAIL https://phabricator.kde.org/D10209 AFFECTED FILES