<div dir="ltr"><div dir="ltr">Hi Povilas,<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Aug 26, 2022 at 12:04 AM Povilas Kanapickas <<a href="mailto:povilas@radix.lt">povilas@radix.lt</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Tomaž,<br>
<br>
I can do either approach as you please. The reason why I created<br>
separate event types for each gesture type was the experience when<br>
implementing touchpad gestures for the X server. Putting all data into a<br>
single struct becomes hard to manage in the end, because it's not<br>
obvious what data each logical event carries and as consequence code<br>
becomes error prone and harder to refactor.<br></blockquote><div><br></div><div>Right, this is always an issue, but then I thought we could do something inside GestureEvent to make this easier (some data class hierarchy for each gesture or something like that).<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Please let me know if I should just put everything into the Gesture<br>
class though. You know better which approach will be easier to maintain<br>
in the specific case of LibreOffice project.<br></blockquote><div> </div><div>I merged those patches, so I guess I'm fine with that. :) There are already separate LongPress and Swipe events too. We can always change it later - it's not a big deal. Thanks.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks a lot,<br>
Povilas<br></blockquote><div><br></div><div>Rest regards, <br></div><div>Tomaž Vajngerl <br></div></div></div>