Skip to content

Commit

Permalink
chore: Increase initial delay to show loading indicator (#14984) (#15319
Browse files Browse the repository at this point in the history
)

The current initial delay to show loading indicator seems quite short. If one deploys an app to another continent, even no op visits can make the indicator "flash". This may kind of just increases the feeling of "broken app". Especially as the indicator in latest versions is bold visually.

Still, at least mine app feels perfectly usable and don't even realise there is a server visit. I don't have the skills to me the indication look less bolder, so I'd just propose to increase the delay a bit. With this level (450ms) all seems to work fine.

Co-authored-by: Matti Tahvonen <[email protected]>
  • Loading branch information
vaadin-bot and mstahv authored Nov 30, 2022
1 parent 2630165 commit 4ca92f6
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
public class LoadingIndicatorConfigurationMap extends NodeMap
implements LoadingIndicatorConfiguration {
public static final String FIRST_DELAY_KEY = "first";
public static final int FIRST_DELAY_DEFAULT = 300;
public static final int FIRST_DELAY_DEFAULT = 450;
public static final String SECOND_DELAY_KEY = "second";
public static final int SECOND_DELAY_DEFAULT = 1500;
public static final String THIRD_DELAY_KEY = "third";
Expand Down

0 comments on commit 4ca92f6

Please sign in to comment.