From bb0d6f580dbf3910ea451b5444a8d68c01425b60 Mon Sep 17 00:00:00 2001
From: Michael Carlberg <c@rlberg.se>
Date: Fri, 23 Dec 2016 22:39:59 +0100
Subject: [PATCH] fix(xkeyboard): Wrap click handler around contents

---
 include/modules/xkeyboard.hpp |  1 +
 src/modules/xkeyboard.cpp     | 21 ++++++++++++++++++---
 2 files changed, 19 insertions(+), 3 deletions(-)

diff --git a/include/modules/xkeyboard.hpp b/include/modules/xkeyboard.hpp
index 1bd772ef..d4534c86 100644
--- a/include/modules/xkeyboard.hpp
+++ b/include/modules/xkeyboard.hpp
@@ -25,6 +25,7 @@ namespace modules {
    public:
     explicit xkeyboard_module(const bar_settings& bar, string name_);
 
+    string get_output();
     void update();
     bool build(builder* builder, const string& tag) const;
 
diff --git a/src/modules/xkeyboard.cpp b/src/modules/xkeyboard.cpp
index e709f130..6e4f57c5 100644
--- a/src/modules/xkeyboard.cpp
+++ b/src/modules/xkeyboard.cpp
@@ -76,15 +76,30 @@ namespace modules {
     broadcast();
   }
 
+  /**
+   * Build module output and wrap it in a click handler use
+   * to cycle between configured layout groups
+   */
+  string xkeyboard_module::get_output() {
+    string output{module::get_output()};
+
+    if (m_keyboard && m_keyboard->size() > 1) {
+      m_builder->cmd(mousebtn::LEFT, EVENT_SWITCH);
+      m_builder->append(output);
+      m_builder->cmd_close();
+    } else {
+      m_builder->append(output);
+    }
+
+    return m_builder->flush();
+  }
+
   /**
    * Map format tags to content
    */
   bool xkeyboard_module::build(builder* builder, const string& tag) const {
     if (tag == TAG_LABEL_LAYOUT) {
-      bool precond{m_keyboard && m_keyboard->size() > 1};
-      builder->cmd(mousebtn::LEFT, EVENT_SWITCH, precond);
       builder->node(m_layout);
-      builder->cmd_close(precond);
     } else if (tag == TAG_LABEL_INDICATOR) {
       size_t n{0};
       for (auto&& indicator : m_indicators) {