D14939: Making more recommended xplanet changes and fixing bugs:

[Jasem Mutlaq]

mutlaqja added inline comments.

INLINE COMMENTS

> pino wrote in xplanetimageviewer.cpp:200-206
> As I mentioned already in the previous review, I'm pretty sure kstars knows about these names already. Isn't it possible to get them from KStarsData, instead of duplicating them?

No it's not possible.

> pino wrote in xplanetimageviewer.cpp:257-259
> As I mentioned already, hardcoding these attributes is a bad UI. Please remove these three lines in all the buttons, let the style decide (following user preferences, which include accessibility).

How do you make them look like icon buttons and not just some button with a pixmap on it?

REPOSITORY
  R321 KStars

REVISION DETAIL
  https://phabricator.kde.org/D14939

To: lancaster, yurchor, pino, mutlaqja, kde-edu
Cc: kde-edu, mutlaqja, pino, yurchor, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20180821/5a09cb34/attachment.html>