Hi Joshua On Wed, Jul 05, 2017 at 08:58:51AM -0500, Joshua Watt wrote: > On Sat, 2017-06-24 at 16:03 -0500, Joshua Watt wrote: > > Previously, the hide_input_panel and show_input_panel messages for > > the text > > input protocol were limited to specific cases, such as showing the > > panel on > > activation, or making the panel visible after activation. Now, > > clients are > > allowed to toggle the panel visiblity at will as long as they are the > > currently > > active client > > > > Signed-off-by: Joshua Watt <jpewhac...@gmail.com> > > --- > > compositor/text-backend.c | 22 ++++++++++++---------- > > 1 file changed, 12 insertions(+), 10 deletions(-) > > > Ping?
I have tested your two patches locally and can confirm that they compile and run. Letting the user toggle the visibility of the input panel with a click (in the --click-to-show case) seems like a good change to me. I doubt that there are any strong opinions on this though... Cheers, Silvan _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel