Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement view for an info.pedia page #39

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nguoiyoujie
Copy link
Contributor

Referencing an old PR #28 , now updated for the current code.

In addition to the features mentioned by that PR, here are the additional improvements.

  • You no longer need to be logged in to be able to view the page

  • The WikiLive hosts each section as its own live view. This makes it easier to be modified or appended. (I intend to add a short description of the major mechanics of the game)

  • The resource categories are now changed to suit a descriptive purpose more than their structure. Flavor texts are considered placeholder and may be changed.
    image

  • The buildable tooltip is its own separate live view. This view is used in the wiki, but it can also be used in city_view to simply code.
    image

  • The resource and buildable list are sectioned with the <details> tag, and can be collapsed.
    image

  • I sourced an icon for education, from here: https://www.svgrepo.com/svg/89864/open-book You may use a different icon if you wish.
    image
    image

  • Supporting modules have been added to separate specific configurations from the view. This means wiki_live (and city_live) can fetch them and not need to maintain their own copies of their definitions.
    city_stock_display is not yet used, but is intended for use with the different forms of stock display on city_view
    image

This PR does not edit city_view nor any of the city calculations, and should not affect them.

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.

None yet

1 participant