Spark moves too fast / hardly visible #189


  • New
  • Defect
Closed
Assigned to nevcairiel
  • _ForgeUser2821998 created this issue Sep 19, 2012

    What steps will reproduce the problem?
    1. Use the tradeskill module of quarts
    2. Do some skilling
    3. Do some normal casts

    What is the expected output? What do you see instead?
    The spark should run from the left side of the bar to the right for each item you create - instead it runs out of the bar at the right.

    The sparks of the the skilltrade bar and the normal cast bar should look the same - instead the one on the normal cast bar is hardly visible.

    What version of the product are you using?
    3.1.1 with the changes from ticket 188. The spark running out of the bar has been observed before the patch of ticket 188 was written.

    Do you have an error log of what happened?
    No error occurs.

    Please provide any additional information below.
    The spark running out of the bar is inherent to the change where the icon is now inside the bar. Change lines 49 and 70 of Tradeskill.lua (where castBarSpark:SetPoint(...) is called) to use castBar:GetWidth() for the width instead of Player.db.profile.w .

    The spark nearly disappearing (which is really obvious if you skill before and after applying this little fix) I expect to be due to texture layering.

  • _ForgeUser2821998 added the tags New Defect Sep 19, 2012
  • LordNarfCDXX posted a comment Nov 4, 2012

    "Please provide any additional information below."

    Change lines 49 and 70 of Tradeskill.lua (where castBarSpark:SetPoint(...) is called) to use castBar:GetWidth() for the width instead of Player.db.profile.w .

    Might I get that dumbed down a bit?

  • _ForgeUser2821998 posted a comment Nov 6, 2012

    Sure.

    1) Open modules/Tradeskill.lua
    2) Search for ":SetPoint" (without the quotes, of course)
    3) At each line you find that (that'll be lines 49 and 70), replace "Player.db.profile.w" with "castBar:GetWidth()"

    Be sure to make a backup of the file, first, in case you make a mistake.


    Edited Nov 6, 2012
  • nevcairiel posted a comment Oct 17, 2014

    Closing pre 6.0 issues


To post a comment, please login or register a new account.