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

improvements suggestions #8

Open
killerwife opened this issue Jan 29, 2018 · 4 comments
Open

improvements suggestions #8

killerwife opened this issue Jan 29, 2018 · 4 comments

Comments

@killerwife
Copy link
Collaborator

  1. add possibility to read spell_template from SQL which includes all custom changes for development - most important in TBC and WOTLK where blizzard removed a plethora of spells
  2. add possibility for interrupt flags enum AND hover over them to display which one it has (found some mistakes in them)
  3. extend showing what values for each effect do, we know what the values do for 90% of effects at this point, I have no problem with providing that info
  4. add commandline option to directly launch given expansion, so its possible to make shortcut to given version without clicking anything, old QSW worked instantly without setup
  5. show reagents needed for a particular spell, also show area needed for particular spell
@sidsukana
Copy link
Owner

sidsukana commented Mar 1, 2018

@killerwife, try 2.33 version

  1. Done
  2. Done
  3. Done

3 and 5, i want rework output template, any ideas?)
What you think about current <details><summary> composition for massive detail info? useful?

@killerwife
Copy link
Collaborator Author

I dont know what you mean by "

".
I looked over the changes, the serversides reading is amazing. I have a few notes on this front. It is clear I need to somehow discern 3 types of spells. 1st type is normal clientside spell. 2nd type is serverside taken from vanilla/tbc patches where it was fully available, and 3rd type, where its purely guesswork. If possible, you know the cmangos-only column IsServerSide, can you add output to UI where if the value is 1, it will show "serverside" and value 2 "serverside-guesswork"?

When it comes to output template, you need to configure defaults better in my opinion.
Examples:

Category: 0 - this is completely useless for example
Chance: 0, Charges: 0 - isnt very useful either, since its customary for very few spells, and so on.
On the other hand stuff like Mechanic: 0 (MECHANIC_NONE) is always useful to know

Another improvement, is to make a setting for Attributes always to be expanded. It is kinda annoying to browse through and have to open it up every time when researching an attribute. The expandable menus look amazing though and improve readability by a lot.

Effects:
Many effects values have special meaning. For example SPELL_EFFECT_SUMMON - 28 Misc Value is super tricky. Same goes for meaning of "damage" value. For SPELL_EFFECT_SCHOOL_DAMAGE its damage, but for lets say knockback, its the horizontal velocity of the knockback effect.

Then there are some columns like AreaID, or MaxAffectedTargets or DmgMultiplier which you don't read currently. (and this is just TBC :D)

@AnonXS
Copy link

AnonXS commented Sep 19, 2018

It seems to be only able to load spellname and description from enGB data folder not any other language. (mpq directory selection)

@killerwife
Copy link
Collaborator Author

killerwife commented Nov 10, 2018

@sidsukana new enums

https://drive.google.com/open?id=1ub0cc9_IgkKpT73yPbPKADQ8lmuT4PX6

has missing comma at TBC and WOTLK jsons, fix it in case u incorporate it

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

No branches or pull requests

3 participants