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

Gear #6

Open
wants to merge 36 commits into
base: master
Choose a base branch
from
Open

Gear #6

wants to merge 36 commits into from

Conversation

darthmaim
Copy link
Owner

@darthmaim darthmaim commented May 30, 2017

This is a WIP, do not merge yet.

  • Change armor itemstats and rarity
  • Change weapon itemstats and rarity
  • Change trinket itemstats and rarity
  • Show correct weapon slots
  • Attribute calculation
  • Responsive
  • "Copy to All" button
  • Import build from API
  • PvP Gear
  • Upgrades/Infusions
  • Translations for attribute names (CritDamage -> Ferocity, …) done in master
  • Tests
Old PR text, before changing to hardcoded attributes

First implementation for the gear panel, still needs lots of work:

  • Include generated gear json file in js bundle or lazy load it later?
    It's always needed anyway, but users will generally look at the above-the-fold content first and we can load it later to reduce the time until the first useful paint. Downside: stat calculation doesn't work until gear is loaded.
  • Upgrade components
  • Responsive
  • QoL: Fill all empty slots with one click/copy upgrade component to all other items
  • Have a nicer dropdown with search and attribute preview
  • Compact the generated json
  • Rebuild json when build changes
  • I don't know why I keep calling stuff "attributeSet", they are called "itemstats" or just "stats" in the API
  • Tests

Repository owner deleted a comment from codecov bot Aug 12, 2017
Repository owner deleted a comment from codecov bot Aug 13, 2017
@darthmaim darthmaim mentioned this pull request Feb 16, 2019
10 tasks
darthmaim and others added 5 commits February 16, 2019 09:50
# Conflicts:
#	src/actions/general.js
#	src/components/StatsOverview/style.css
# Conflicts:
#	src/components/StatsOverview/StatsOverview.jsx
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.

1 participant