We have moved to Github. Please open tickets there.

Opened 2 years ago

Last modified 4 months ago

#3617 new Defect

Review and correct z-index issues in default theme

Reported by: cash Owned by:
Priority: low Milestone: Elgg 1.8.14
Component: UI/UX Version: Github Master
Severity: minor Keywords:
Cc: brett@… Difficulty: easy

Description

I noticed some issues when working with the lightbox. For example, the site menu was actually above the lightbox background (of 1000).

Change History (16)

comment:1 Changed 21 months ago by brettp

  • Milestone changed from Elgg 1.8.0 to Elgg 1.8.1b

comment:2 Changed 21 months ago by cash 

topbar: 9000
system messages: 2000
login dropdown toggle: 100
popup module: 9999
dropdown module (used for login dropdown box): 200
site menu: 1
user hover menu: 10000
lightbox: 1001 for bg, 1100-1104 for lightbox components and throbber

comment:3 Changed 20 months ago by cash

I also added a z-index for the site menu more dropdown as 1

site menu and the more dropdown need to be higher than the content the dropdown covers.

Possible conflicts:

  1. system messages and the login dropdown box
  2. user hover menu and popups/lightbox

comment:4 Changed 20 months ago by cash

  • Milestone changed from Elgg 1.8.1b to Elgg 1.8.1

comment:5 Changed 19 months ago by cash

  • Priority changed from normal to low

comment:6 Changed 19 months ago by cash

  • Milestone changed from Elgg 1.8.1 to Elgg 1.8.2

The user hover needs to be above the lightbox and popup in case an avatar is included in them. If either a user hover or popup is already shown, they are toggled if a lightbox is launched by a click.

Wondering whether popup and lightbox should be at the same level.

The only clear conflict is the system messages and the login box.

I'm pushing this back to 1.8.2 unless there are objections.

comment:7 Changed 18 months ago by cash

Need to create a test page that works through the permutations of this.

comment:8 Changed 17 months ago by cash

  • Milestone changed from Elgg 1.8.2 to Elgg 1.8.3

comment:9 Changed 17 months ago by cash

  • Milestone changed from Elgg 1.8.3 to Elgg 1.8.4

comment:10 Changed 13 months ago by brettp

  • Milestone changed from Elgg 1.8.4 to Elgg 1.8.5

comment:11 Changed 10 months ago by sembrestels

site menu should have more than 1, because it can conflict with page elements. I bet for 100, as login dropdown toggle.

comment:12 Changed 6 months ago by brettp

  • Milestone changed from Elgg 1.8.10 to Elgg 1.8.12

comment:13 Changed 5 months ago by brettp

The only clear conflict is the system messages and the login box.

Do you think system messages should be below the login dropdown box?

comment:14 Changed 5 months ago by Brett Profitt <brett.profitt@…>

In f17c0c674f084c48a0b2919ac8fe7136a819859f/elgg:

Refs #3617. Added misc section for developer's theme preview.

comment:15 Changed 5 months ago by brettp

  • Milestone changed from Elgg 1.8.12 to Elgg 1.8.13

comment:16 Changed 4 months ago by cash

  • Milestone changed from Elgg 1.8.13 to Elgg 1.8.14
Note: See TracTickets for help on using tickets.