Actions
Patch #17401
closedBetter fix for r13159 issue #16708
Start date:
Due date:
% Done:
0%
Estimated time:
Description
You can fix this one with a little bit cleaner code:
Change: <button class="tab-left" onclick="moveTabLeft(this); return false;"></button>
to: <button class="tab-left" type="button" onclick="moveTabLeft(this);"></button>
Browsers defaults often to type="submit" which submits the form. If you explicite define a type, it will be fine.
The return false; is not the cleanest way, as... if you call some code afterwards, it won't be executed because the propagation ends there.
Updated by Jean-Philippe Lang over 10 years ago
- Category changed from UI to Code cleanup/refactoring
- Status changed from New to Closed
- Target version set to 2.6.0
Patch committed in r13326, thanks!
Actions