Project

General

Profile

Actions

Patch #20271

closed

Fix for multiple tabs on the same page

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

Status:
Closed
Priority:
Normal
Category:
UI
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

When navigation tabs use more space than their container (e.g. /settings), tab buttons are displayed in order to horizontally scroll to non visible tabs.

The javascript function (in application.js) displayTabButtons takes care of settting these tab buttons to a visible / non-visible state. This function is theoretically supposed to work for severeal tab navigations. It applies it's logic to all divs, with class .tabs in a for each loop.

To my knowledge redmine does not have pages with several of these tab navigations. If a plugins tries to use this funtionality, though, it will run into the following problem: If one tab navigation meets the condition to be displayed, all other tab navigations on the page (even when not needed) will have the tab buttons for horizontal scrolling enabled.

This patch fixes this false behaviour.


Files


Related issues

Related to Redmine - Patch #20906: Fix mulitple tab navigation highlighting and content hidingClosedJean-Philippe Lang

Actions
Blocks Redmine - Patch #20632: Tab left/right buttons for project menuClosedJean-Philippe Lang

Actions
Actions #1

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

  • Subject changed from Fix for multible tab navigations to Fix for multiple tab navigations
  • Target version set to Candidate for next minor release
Actions #2

Updated by Toshi MARUYAMA over 8 years ago

  • Target version changed from Candidate for next minor release to 3.2.0
Actions #3

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

  • Blocks Patch #20632: Tab left/right buttons for project menu added
Actions #4

Updated by Jean-Philippe Lang over 8 years ago

  • Subject changed from Fix for multiple tab navigations to Fix for multiple tabs on the same page
  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang

Patch committed in r14624, thanks.

Actions #5

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

  • Related to Patch #20906: Fix mulitple tab navigation highlighting and content hiding added
Actions

Also available in: Atom PDF