graesslin marked an inline comment as done.
graesslin added inline comments.

INLINE COMMENTS

> sebas wrote in text-input-unstable-v2.xml:153
> "a URL"

this is an XML file I imported from 3rd party (QtWayland). Not going to touch 
it.

> sebas wrote in registry.h:451
> A small note about the protocol (i.e. which version of the interface should 
> be used) probably wouldn't hurt (as the two creators may end up being 
> confusing). This is a bit high-level though, for APIDOCs, but still raises 
> this question.

Isn't the explanation in createTextInputManager sufficient? After all it says 
one should prefer createTextInputManager

> sebas wrote in textinput.h:81
> Does this actually show the panel then? (Could be more clear what's expected 
> here, what's the relationship between enable/disable and show*/hide*)

honest answer: I don't know.

REPOSITORY
  rKWAYLAND KWayland

BRANCH
  graesslin/text-input

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, sebas, Plasma
Cc: sebas, broulik, plasma-devel
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to