Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#1464 closed bug (fixed (in master))

Spell browser lost its header

Reported by: PowerWyrm Owned by: noz
Milestone: 3.3.0 Keywords: ui
Cc:

Description

In menu_refresh(), the menu header is now displayed before the browse hook is called. With the "browse spell" menu, here's what happens:

  • browsing is initiated by spell_menu_browse()
  • screen_save() is called, which makes a copy of the display
  • menu_select() is called
  • menu_refresh() is called
  • menu header is displayed
  • browser hook is called: spell_menu_browser()
  • screen_load() is called, which reloads the saved copy of the display... without the menu header!
  • spell description is displayed
  • spell list is displayed (spell_menu_display)

Change History (4)

comment:1 Changed 9 years ago by magnate

  • Keywords ui added
  • Milestone changed from Triage to 3.3.0
  • Status changed from new to confirmed

comment:2 Changed 8 years ago by noz

  • Owner set to noz
  • Status changed from confirmed to assigned

comment:3 Changed 8 years ago by takkaria

  • Status changed from assigned to pending

in staging 2b74229

comment:4 Changed 8 years ago by magnate

  • Resolution set to fixed
  • Status changed from pending to closed

In master as of [r5c8f16e]

Note: See TracTickets for help on using tickets.