Dominos

Dominos

19M Downloads

The maximum value of rowoffset should be added with the value of Space.

WhisperWindE opened this issue · 7 comments

commented

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Describe the solution you'd like
A clear and concise description of what you want to happen.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.
This is the only way to achieve vertical alignment when staggered rows.
image

commented

Bar-RowOffset range (0-36)
36 is the width of the button
How to add the width of Spaceing
For example, the default Space=2
The range of offset should be 0----38 (36+2)

commented

I never actually considered this particular bar configuration before

commented
function ButtonBar:SetRowOffset(rowOffset)
    self.sets.rowOffset = rowOffset + self.sets.spacing
    self:Layout()
end

I simply modified the function
to temporarily implement this offset
but there is a problem, every time I switch to modify, this offset will be automatically added

Which one should I modify correctly?

commented

I think you want to modify the layout function directly

commented

This is implemented now - The min and max values of the slider will include spacing. You can also type in a value to bypass the limits that are in place.

commented

Which file was changed? I used it WotLK Classic

commented

I updated Dominos Config’s slider defaults

0850994