Skip to content

feat(152): implement Technology tab in "?" info popup - #1502

Open
tiweb442 wants to merge 3 commits into
lanedirt:mainfrom
tiweb442:feature/152-technology-tab
Open

feat(152): implement Technology tab in "?" info popup#1502
tiweb442 wants to merge 3 commits into
lanedirt:mainfrom
tiweb442:feature/152-technology-tab

Conversation

@tiweb442

@tiweb442 tiweb442 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Implements the Technology tab in the building/unit/research "?" info popup (Implement "?" popup technology tab #152)
  • Shows all game objects grouped by Construction, Research, Ships, Defense, and Rockets, with accordion expand/collapse
  • Renders sprite icons and direct prerequisites with fulfilled/unfulfilled status based on the current player's levels

Test plan

  • Open any building/unit/research "?" popup and switch to the Technology tab
  • Confirm all five category sections are listed and collapsed by default
  • Confirm clicking a category heading toggles its list
  • Confirm technology links open Techinfo and prerequisite links open Techtree
  • Confirm prerequisite rows show green (fulfilled) / red (unfulfilled) based on current levels
  • Run php artisan test --filter=TechtreeTest

Show the global technology list with category accordions and prerequisite status to match official OGame.

Co-authored-by: Cursor <cursoragent@cursor.com>
Match official OGame CSS (display:none) instead of forcing all
category lists open when the tab loads.

Co-authored-by: Cursor <cursoragent@cursor.com>

@Geda173 Geda173 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested it locally and verified both the feature behavior and the test's rigor.

One optional nit for a follow-up (non-blocking): the <a class="prerequisites"> wraps a <ul>, which is invalid HTML nesting. Browsers tolerate it, but worth tidying if it diverges from the surrounding markup conventions.

Replace the ul/li inside a.prerequisites with divs so the markup stays
valid while preserving the official OGame CSS selectors.

Co-authored-by: Cursor <cursoragent@cursor.com>
@tiweb442

Copy link
Copy Markdown
Contributor Author

Thanks @Geda173 — addressed the optional nesting nit in 89ccded: the prerequisites link now uses divs instead of a nested ul/li, so it stays valid while keeping the official a.prerequisites CSS selectors.

@github-actions

Copy link
Copy Markdown

Preview deployment

Preview environment for this PR is ready for functional testing. Use one of the test accounts below to log in.

Status URL Commit Last deployed
✅ Live https://pr-1502.preview.ogamex.dev 89ccded 2026-07-24 21:50:10 UTC
Test accounts (click to expand)
Email (login) Password Role Class Planets Description
test1@ogamex.dev test admin Collector 2 Admin, high resources, mining focus
test2@ogamex.dev test player General 1 Military focus, ships & fleet
test3@ogamex.dev test player Discoverer 1 Exploration, probes & cargo ships
test4@ogamex.dev test player None 1 New player, no class selected yet
test5@ogamex.dev test moderator Collector 1 Moderator, balanced setup
test6@ogamex.dev test player General 1 Vacation mode enabled
test7@ogamex.dev test player General 3 Heavy defense across planets
test8@ogamex.dev test player Discoverer 1 Fleet focus, inactive (i) 8 days
test9@ogamex.dev test player Collector 1 Beginner, long inactive (I) 40 days
test10@ogamex.dev test player General 5 Endgame, 90 unread messages

This preview will be automatically destroyed when the PR is closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants