attempt to fix shrinking sliders
This commit is contained in:
parent
7126cd58a5
commit
d96bbd3a08
|
@ -189,6 +189,21 @@ public class Utils {
|
||||||
return high;
|
return high;
|
||||||
return val;
|
return val;
|
||||||
}
|
}
|
||||||
|
/**
|
||||||
|
* Clamps a value between a lower and upper bound.
|
||||||
|
* @param val the value to clamp
|
||||||
|
* @param low the lower bound
|
||||||
|
* @param high the upper bound
|
||||||
|
* @return the clamped value
|
||||||
|
* @author fluddokt
|
||||||
|
*/
|
||||||
|
public static double clamp(double val, double low, double high) {
|
||||||
|
if (val < low)
|
||||||
|
return low;
|
||||||
|
if (val > high)
|
||||||
|
return high;
|
||||||
|
return val;
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Returns the distance between two points.
|
* Returns the distance between two points.
|
||||||
|
|
|
@ -209,9 +209,9 @@ public class Slider extends GameObject {
|
||||||
final int fadeInTime = game.getFadeInTime();
|
final int fadeInTime = game.getFadeInTime();
|
||||||
float scale = timeDiff / (float) approachTime;
|
float scale = timeDiff / (float) approachTime;
|
||||||
float approachScale = 1 + scale * 3;
|
float approachScale = 1 + scale * 3;
|
||||||
float fadeinScale = (timeDiff - approachTime + fadeInTime) / (float) fadeInTime;
|
double fadeinScale = (timeDiff - approachTime + fadeInTime) / (double) fadeInTime;
|
||||||
float alpha = Utils.clamp(1 - fadeinScale, 0, 1);
|
float alpha = Utils.clamp(1 - (float) fadeinScale, 0, 1);
|
||||||
float decorationsAlpha = Utils.clamp(-2.0f * fadeinScale, 0, 1);
|
float decorationsAlpha = Utils.clamp(-2.0f * (float) fadeinScale, 0, 1);
|
||||||
boolean overlayAboveNumber = Options.getSkin().isHitCircleOverlayAboveNumber();
|
boolean overlayAboveNumber = Options.getSkin().isHitCircleOverlayAboveNumber();
|
||||||
float oldAlpha = Colors.WHITE_FADE.a;
|
float oldAlpha = Colors.WHITE_FADE.a;
|
||||||
Colors.WHITE_FADE.a = color.a = alpha;
|
Colors.WHITE_FADE.a = color.a = alpha;
|
||||||
|
@ -225,7 +225,7 @@ public class Slider extends GameObject {
|
||||||
}
|
}
|
||||||
|
|
||||||
curveColor.a = curveAlpha;
|
curveColor.a = curveAlpha;
|
||||||
boolean isCurveCompletelyDrawn = drawSliderTrack(trackPosition, alpha);
|
boolean isCurveCompletelyDrawn = drawSliderTrack(trackPosition, Utils.clamp(1d - fadeinScale, 0d, 1d));
|
||||||
color.a = alpha;
|
color.a = alpha;
|
||||||
|
|
||||||
g.pushTransform();
|
g.pushTransform();
|
||||||
|
@ -401,40 +401,39 @@ public class Slider extends GameObject {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
private boolean drawSliderTrack(int trackPosition, float snakingSliderProgress) {
|
private boolean drawSliderTrack(int trackPosition, double snakingSliderProgress) {
|
||||||
float curveIntervalTo = Options.isSliderSnaking() ? snakingSliderProgress : 1f;
|
double curveIntervalTo = Options.isSliderSnaking() ? snakingSliderProgress : 1d;
|
||||||
float curveIntervalFrom = 0f;
|
double curveIntervalFrom = 0d;
|
||||||
if (Options.isShrinkingSliders()) {
|
if (Options.isShrinkingSliders()) {
|
||||||
float sliderprogress = (trackPosition - getTime() - (sliderTime * (repeats - 1))) / sliderTime;
|
double sliderprogress = (trackPosition - getTime() - ((double) sliderTime * (repeats - 1))) / (double) sliderTime;
|
||||||
if (sliderprogress > 0) {
|
if (sliderprogress > 0) {
|
||||||
curveIntervalFrom = sliderprogress;
|
curveIntervalFrom = sliderprogress;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
int curvelen = curve.getCurvePoints().length;
|
||||||
if (Options.isMergingSliders()) {
|
if (Options.isMergingSliders()) {
|
||||||
if (Options.isShrinkingSliders() && curveIntervalFrom > 0) {
|
if (Options.isShrinkingSliders() && curveIntervalFrom > 0) {
|
||||||
int curvelen = curve.getCurvePoints().length;
|
|
||||||
if (repeats % 2 == 0) {
|
if (repeats % 2 == 0) {
|
||||||
game.spliceSliderCurve(baseSliderFrom + (int) ((1f - curveIntervalFrom) * curvelen), baseSliderFrom + curvelen);
|
game.spliceSliderCurve(baseSliderFrom + (int) ((1d - curveIntervalFrom) * curvelen), baseSliderFrom + curvelen + 1);
|
||||||
} else {
|
} else {
|
||||||
game.setSlidercurveFrom(baseSliderFrom + (int) (curveIntervalFrom * curvelen) + 1);
|
game.setSlidercurveFrom(baseSliderFrom + (int) (curveIntervalFrom * curvelen));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
game.setSlidercurveTo(baseSliderFrom + (int) (curveIntervalTo * curve.getCurvePoints().length));
|
game.setSlidercurveTo(baseSliderFrom + (int) (curveIntervalTo * curve.getCurvePoints().length));
|
||||||
} else {
|
} else {
|
||||||
if (Options.isFallbackSliders() && curveIntervalFrom > 0 && repeats % 2 == 0) {
|
if (Options.isFallbackSliders() && curveIntervalFrom > 0 && repeats % 2 == 0) {
|
||||||
curve.draw(curveColor, 1f - curveIntervalTo, 1f - curveIntervalFrom);
|
curve.draw(curveColor, (int) (curveIntervalFrom * curvelen), (int) (curveIntervalTo * curvelen));
|
||||||
} else {
|
} else {
|
||||||
if (Options.isShrinkingSliders() && curveIntervalFrom > 0 && !Options.isFallbackSliders()) {
|
if (Options.isShrinkingSliders() && curveIntervalFrom > 0 && !Options.isFallbackSliders()) {
|
||||||
int curvelen = curve.getCurvePoints().length;
|
|
||||||
if (repeats % 2 == 0) {
|
if (repeats % 2 == 0) {
|
||||||
curve.splice((int) ((1f - curveIntervalFrom) * curvelen), curvelen);
|
curve.splice((int) ((1d - curveIntervalFrom) * curvelen), curvelen);
|
||||||
curveIntervalFrom = 0f;
|
curveIntervalFrom = 0d;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
curve.draw(curveColor, curveIntervalFrom, curveIntervalTo);
|
curve.draw(curveColor, (int) (curveIntervalFrom * curvelen), (int) (curveIntervalTo * curvelen));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return curveIntervalTo == 1f;
|
return curveIntervalTo == 1d;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
|
@ -121,32 +121,27 @@ public abstract class Curve {
|
||||||
* Draws the full curve to the graphics context.
|
* Draws the full curve to the graphics context.
|
||||||
* @param color the color filter
|
* @param color the color filter
|
||||||
*/
|
*/
|
||||||
public void draw(Color color) { draw(color, 0f, 1f); }
|
public void draw(Color color) { draw(color, 0, curve.length); }
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Draws the curve in the range [0, t] (where the full range is [0, 1]) to the graphics context.
|
* Draws the curve in the range [0, t] (where the full range is [0, 1]) to the graphics context.
|
||||||
* @param color the color filter
|
* @param color the color filter
|
||||||
* @param t1 interval to draw from
|
* @param from index to draw from
|
||||||
* @param t2 interval to draw to
|
* @param to index to draw to (exclusive)
|
||||||
*/
|
*/
|
||||||
public void draw(Color color, float t1, float t2) {
|
public void draw(Color color, int from, int to) {
|
||||||
if (curve == null)
|
if (curve == null)
|
||||||
return;
|
return;
|
||||||
|
|
||||||
t1 = Utils.clamp(t1, 0f, 1f);
|
|
||||||
t2 = Utils.clamp(t2, 0f, 1f);
|
|
||||||
|
|
||||||
// peppysliders
|
// peppysliders
|
||||||
if (Options.isFallbackSliders() || Options.getSkin().getSliderStyle() == Skin.STYLE_PEPPYSLIDER || !mmsliderSupported) {
|
if (Options.isFallbackSliders() || Options.getSkin().getSliderStyle() == Skin.STYLE_PEPPYSLIDER || !mmsliderSupported) {
|
||||||
int drawFrom = (int) (curve.length * t1);
|
|
||||||
int drawUpTo = (int) (curve.length * t2);
|
|
||||||
Image hitCircle = GameImage.HITCIRCLE.getImage();
|
Image hitCircle = GameImage.HITCIRCLE.getImage();
|
||||||
Image hitCircleOverlay = GameImage.HITCIRCLE_OVERLAY.getImage();
|
Image hitCircleOverlay = GameImage.HITCIRCLE_OVERLAY.getImage();
|
||||||
for (int i = drawFrom; i < drawUpTo; i++)
|
for (int i = from; i < to; i++)
|
||||||
hitCircleOverlay.drawCentered(curve[i].x, curve[i].y, Colors.WHITE_FADE);
|
hitCircleOverlay.drawCentered(curve[i].x, curve[i].y, Colors.WHITE_FADE);
|
||||||
float a = fallbackSliderColor.a;
|
float a = fallbackSliderColor.a;
|
||||||
fallbackSliderColor.a = color.a;
|
fallbackSliderColor.a = color.a;
|
||||||
for (int i = drawFrom; i < drawUpTo; i++)
|
for (int i = from; i < to; i++)
|
||||||
hitCircle.drawCentered(curve[i].x, curve[i].y, fallbackSliderColor);
|
hitCircle.drawCentered(curve[i].x, curve[i].y, fallbackSliderColor);
|
||||||
fallbackSliderColor.a = a;
|
fallbackSliderColor.a = a;
|
||||||
}
|
}
|
||||||
|
@ -155,7 +150,7 @@ public abstract class Curve {
|
||||||
else {
|
else {
|
||||||
if (renderState == null)
|
if (renderState == null)
|
||||||
renderState = new CurveRenderState(hitObject, curve);
|
renderState = new CurveRenderState(hitObject, curve);
|
||||||
renderState.draw(color, borderColor, t1, t2);
|
renderState.draw(color, borderColor, from, to);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -89,7 +89,7 @@ public class CurveRenderState {
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Undo the static state. Static state setup caused by calls to
|
* Undo the static state. Static state setup caused by calls to
|
||||||
* {@link #draw(org.newdawn.slick.Color, org.newdawn.slick.Color, float, float)}
|
* {@link #draw(org.newdawn.slick.Color, org.newdawn.slick.Color, int, int)}
|
||||||
* are undone.
|
* are undone.
|
||||||
*/
|
*/
|
||||||
public static void shutdown() {
|
public static void shutdown() {
|
||||||
|
@ -129,17 +129,13 @@ public class CurveRenderState {
|
||||||
* runs it just draws the cached copy to the screen.
|
* runs it just draws the cached copy to the screen.
|
||||||
* @param color tint of the curve
|
* @param color tint of the curve
|
||||||
* @param borderColor the curve border color
|
* @param borderColor the curve border color
|
||||||
* @param t2 the point up to which the curve should be drawn (in the interval [0, 1])
|
* @param from index to draw from
|
||||||
|
* @param to index to draw to (exclusive)
|
||||||
*/
|
*/
|
||||||
public void draw(Color color, Color borderColor, float t1, float t2) {
|
public void draw(Color color, Color borderColor, int from, int to) {
|
||||||
t1 = Utils.clamp(t1, 0.0f, 1.0f);
|
|
||||||
t2 = Utils.clamp(t2, 0.0f, 1.0f);
|
|
||||||
float alpha = color.a;
|
float alpha = color.a;
|
||||||
|
|
||||||
int drawFrom = (int) (t1 * curve.length);
|
if (lastPointDrawn != to || firstPointDrawn != from) {
|
||||||
int drawUpTo = (int) (t2 * curve.length);
|
|
||||||
|
|
||||||
if (lastPointDrawn != drawUpTo || firstPointDrawn != drawFrom) {
|
|
||||||
int oldFb = GL11.glGetInteger(EXTFramebufferObject.GL_FRAMEBUFFER_BINDING_EXT);
|
int oldFb = GL11.glGetInteger(EXTFramebufferObject.GL_FRAMEBUFFER_BINDING_EXT);
|
||||||
int oldTex = GL11.glGetInteger(GL11.GL_TEXTURE_BINDING_2D);
|
int oldTex = GL11.glGetInteger(GL11.GL_TEXTURE_BINDING_2D);
|
||||||
//glGetInteger requires a buffer of size 16, even though just 4
|
//glGetInteger requires a buffer of size 16, even though just 4
|
||||||
|
@ -148,20 +144,20 @@ public class CurveRenderState {
|
||||||
GL11.glGetInteger(GL11.GL_VIEWPORT, oldViewport);
|
GL11.glGetInteger(GL11.GL_VIEWPORT, oldViewport);
|
||||||
EXTFramebufferObject.glBindFramebufferEXT(EXTFramebufferObject.GL_FRAMEBUFFER_EXT, fbo.getID());
|
EXTFramebufferObject.glBindFramebufferEXT(EXTFramebufferObject.GL_FRAMEBUFFER_EXT, fbo.getID());
|
||||||
GL11.glViewport(0, 0, fbo.width, fbo.height);
|
GL11.glViewport(0, 0, fbo.width, fbo.height);
|
||||||
if (lastPointDrawn <= 0 || lastPointDrawn > drawUpTo) {
|
if (lastPointDrawn <= 0 || lastPointDrawn > to) {
|
||||||
lastPointDrawn = 0;
|
lastPointDrawn = 0;
|
||||||
GL11.glClearColor(0.0f, 0.0f, 0.0f, 0.0f);
|
GL11.glClearColor(0.0f, 0.0f, 0.0f, 0.0f);
|
||||||
GL11.glClear(GL11.GL_COLOR_BUFFER_BIT | GL11.GL_DEPTH_BUFFER_BIT);
|
GL11.glClear(GL11.GL_COLOR_BUFFER_BIT | GL11.GL_DEPTH_BUFFER_BIT);
|
||||||
}
|
}
|
||||||
if (firstPointDrawn != drawFrom) {
|
if (firstPointDrawn != from) {
|
||||||
GL11.glClearColor(0.0f, 0.0f, 0.0f, 0.0f);
|
GL11.glClearColor(0.0f, 0.0f, 0.0f, 0.0f);
|
||||||
GL11.glClear(GL11.GL_COLOR_BUFFER_BIT | GL11.GL_DEPTH_BUFFER_BIT);
|
GL11.glClear(GL11.GL_COLOR_BUFFER_BIT | GL11.GL_DEPTH_BUFFER_BIT);
|
||||||
firstPointDrawn = drawFrom;
|
this.renderCurve(color, borderColor, from, to, true);
|
||||||
this.renderCurve(color, borderColor, drawFrom, drawUpTo, true);
|
|
||||||
} else {
|
} else {
|
||||||
this.renderCurve(color, borderColor, lastPointDrawn, drawUpTo, false);
|
this.renderCurve(color, borderColor, lastPointDrawn, to, false);
|
||||||
}
|
}
|
||||||
lastPointDrawn = drawUpTo;
|
lastPointDrawn = to;
|
||||||
|
firstPointDrawn = from;
|
||||||
color.a = 1f;
|
color.a = 1f;
|
||||||
|
|
||||||
GL11.glBindTexture(GL11.GL_TEXTURE_2D, oldTex);
|
GL11.glBindTexture(GL11.GL_TEXTURE_2D, oldTex);
|
||||||
|
|
|
@ -329,7 +329,7 @@ public class Game extends BasicGameState {
|
||||||
int obj = objectIndex;
|
int obj = objectIndex;
|
||||||
while (obj < gameObjects.length) {
|
while (obj < gameObjects.length) {
|
||||||
if (gameObjects[obj] instanceof Slider) {
|
if (gameObjects[obj] instanceof Slider) {
|
||||||
slidercurveFrom = slidercurveTo = (float) ((Slider) gameObjects[obj]).baseSliderFrom / knorkesliders.getCurvePoints().length;
|
slidercurveFrom = slidercurveTo = ((Slider) gameObjects[obj]).baseSliderFrom;
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
obj++;
|
obj++;
|
||||||
|
@ -1582,17 +1582,15 @@ public class Game extends BasicGameState {
|
||||||
GameMod.loadModState(previousMods);
|
GameMod.loadModState(previousMods);
|
||||||
}
|
}
|
||||||
|
|
||||||
private float slidercurveFrom;
|
private int slidercurveFrom;
|
||||||
private float slidercurveTo;
|
private int slidercurveTo;
|
||||||
|
|
||||||
public void setSlidercurveFrom(int slidercurveFrom) {
|
public void setSlidercurveFrom(int slidercurveFrom) {
|
||||||
float pos = (float) slidercurveFrom / knorkesliders.getCurvePoints().length;
|
this.slidercurveFrom = Math.max(slidercurveFrom, this.slidercurveFrom);
|
||||||
this.slidercurveFrom = Math.max(pos, this.slidercurveFrom);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
public void setSlidercurveTo(int slidercurveTo) {
|
public void setSlidercurveTo(int slidercurveTo) {
|
||||||
float pos = (float) slidercurveTo / knorkesliders.getCurvePoints().length;
|
this.slidercurveTo = Math.max(slidercurveTo, this.slidercurveTo);
|
||||||
this.slidercurveTo = Math.max(pos, this.slidercurveTo);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
public void spliceSliderCurve(int from, int to) {
|
public void spliceSliderCurve(int from, int to) {
|
||||||
|
|
Loading…
Reference in New Issue
Block a user