I like the readability of my way better. you don't have to go all the way to the element inside braskets to understand it.
people would usually skip "variable = xxx", if it does nothing.
you can easily understand what is tap(next) than tap(self.marionette.find_element(*self._next_button_locator_1))