Project

General

Profile

Actions

Patch #20906

closed

Fix mulitple tab navigation highlighting and content hiding

Added by Felix Gliesche over 8 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Normal
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Assuming you have multiple tab navigations on one page, clicking on a tab leads to the false behaviour that highlighted tabs in another tab navigation lose their highlighting. This is caused by the current implementation of showTab in application.js, because it only assumes the case of having only one tab navigation on one page (see source:trunk/public/javascripts/application.js#L319). This assumption also causes the behaviour, that all elements that match the selector

div#content .tab-content

will be hidden if one tab is clicked (see source:trunk/public/javascripts/application.js#L318).

The attached patch takes the possibility of having several tab navigations into consideration. It does not remove the highlighting for other tab navigations and hides the tab content of the corresponding tab context.


Files


Related issues

Related to Redmine - Patch #20271: Fix for multiple tabs on the same pageClosedJean-Philippe Lang

Actions
Actions #1

Updated by Jan from Planio www.plan.io over 8 years ago

  • Target version set to Candidate for next minor release
Actions #2

Updated by Jan from Planio www.plan.io over 8 years ago

  • Related to Patch #20271: Fix for multiple tabs on the same page added
Actions #3

Updated by Jean-Philippe Lang over 8 years ago

  • Status changed from New to Resolved
  • Assignee set to Jean-Philippe Lang
  • Target version changed from Candidate for next minor release to 3.1.2

Committed, thanks.

Actions #4

Updated by Jean-Philippe Lang over 8 years ago

  • Category changed from UI to Code cleanup/refactoring
  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF