This broke after upgrading deps and couldn't figure it out with a quick look.
It just makes it a bit more convenient when adding new pages & debugging
but not really that necessary so decided to delete it for now.
* Initial
* Pass mainWeaponId
* Repeating args to "context"
* Fix todos
* Date header
* Fix comparison showing for sub weapon def dmg even if no abilities selected
* Sub damage table charts
* Chart dot focus style
* Color grid initial
* Can change shots in the grid
* Optimize perf
* Style - cells
* Grid hover effect
* Mobile friendly
* Build filters WIP
* Move builds to feature folder
* Move stuff to builds feature folder
* Working filters
* AP values as dropdown
* Should revalidate logic
* Remove debounce
* Fix main ability filter not working by default
* Persist filters to search params
* Disable button if too many filters
* Stack on mobile
* Abilities in i18n json
* Ability icon in filter
* Add i18n
* E2E test
* Remove done todo
* Remove light mode
* Trim header
* New front page initial
* Get rid of build layout
* Breadcrumbs
* Desktop side nav
* Overhaul colors
* Add breadcrumbs
* New sub nav style
* Front page action buttons
* Add back add new build button
* Add articles page with icon
* Minor Object damage page layout tweaks
* Remove one unnecessary render from object damage
* Fix wrong link in article page
* Profile -> My Page in header
* Log in/out buttons in front
* Add drawings to front page
* Remove unnecessary comment
* Issue 1036: non-draggable ability icons will no longer have its cursor morphed to a pointing hand icon
* Revert "Issue 1036: non-draggable ability icons will no longer have its cursor morphed to a pointing hand icon"
This reverts commit a0de4e0047.
* Issue 1036: Non-draggable ability icons will no longer have its cursor morphed to a pointing hand icon.
- Done correctly by conditionally choosing to render a <button /> or <div /> HTML element for ability icons everywhere Ability() was invoked by passing in readonly
* Removed stray console.warn() line, added a brief comment
* Removed unnecessary tabIndex adjustment for AbilityTag element if it is readonly (since it is no longer rendered as a button if it is readonly)
- We now also condtionally give type="button" to AbilityTag only if it is actually a button
* Conditionally set AbilitiesSelector's Ability JSX elements to readonly if the ability is "UNKNOWN"
* Fixed failed check for Prettier in CI pipeline
* Ability component now only chooses the readonly class if no onClick function is passed into it
- Exception: if the ability is "UNKNOWN", it will be forced to readonly
* Fixed Prettier error in CI pipeline
* Fixed Typecheck error in BuildCard.tsx
* Fixed Prettier CI pipeline error again
* Was checking for an undefined type incorrectly. Now the logic works properly
* Fixed Prettier error
* Cleaned up classname declaration syntax to React's shorthand
- Do not allow dragging an ability onto a slot that cannot accept that
ability.
- When dragging, dim slots that are not a valid target for the ability
currently being dragged.
- Do not make any changes to the existing click behavior to assign
abilities.
`<Ability/>` can now be tabbed through, and it now shares the same focus
outline, hover, and mousedown styles with `<AbilitiesSelector/>`. This
includes the `cursor: pointer` style that makes it more obvious that
this component is clickable like other buttons.
Fixes#894.
