Opened 2 years ago

Closed 2 years ago

#6484 closed defect (fixed)

Data counter clashes with tech icon in top right corner

Reported by: Langbart Owned by: phosit
Priority: Release Blocker Milestone: Alpha 26
Component: UI – In-game Keywords:
Cc: Patch: Phab:D4585

Description (last modified by Langbart)

to reproduce

Issue (1)

  • start a game with the default game settings
  • Select the CC and research something
  • press hotkey to enable FPS overlay
  • The tech icon disappears
  • See GIF below for more clarity

Issue (2)

  • enable FPS from the options settings not with the hotkey (If you enable the FPS with the hotkey and go into the option settings, you would expect it to be saved, but no, the next time you restart the app, the FPS is off again.)
  • start a game
  • Select the CC and research something
  • data counter is always above the tech icon
  • See GIF below for more clarity

bisect

[26523]

reproducible

Issue (1) is not reproducible on A25b.
Issue (2) is reproducible to a certain extent, but as soon as you trigger an additional hotkey (e.g. elapsed time) the tech icon moves below the data Counter. This is currently not the case while testing it on the latest GIT version [26735].

Attachments (2)

issue_1.gif (42.5 KB ) - added by Langbart 2 years ago.
issue_2.gif (74.3 KB ) - added by Langbart 2 years ago.

Download all attachments as: .zip

Change History (7)

by Langbart, 2 years ago

Attachment: issue_1.gif added

by Langbart, 2 years ago

Attachment: issue_2.gif added

comment:1 by Langbart, 2 years ago

Component: Core engineUI – In-game
Description: modified (diff)
Priority: Should HaveRelease Blocker

bisect

comment:2 by Langbart, 2 years ago

Description: modified (diff)

Clarification

comment:3 by phosit, 2 years ago

Owner: set to phosit

comment:4 by phosit, 2 years ago

Patch: Phab:D4585

comment:5 by Freagarach, 2 years ago

Resolution: fixed
Status: newclosed

In 26755:

Fix research icon position.

It was overlayed with the FPS overlay which rendered it unreadable.

Introduced in r26523 / rP26523
Reported by: @Langbart
Patch by: @phosit
Differential revision: https://code.wildfiregames.com/D4585
Comments by: @elexis
Fixes #6484

Note: See TracTickets for help on using tickets.