fix: prevent the reset button from clearing the leader board
This commit is contained in:
parent
2dfa43f157
commit
c5e4c6d27a
1 changed files with 5 additions and 9 deletions
|
|
@ -104,15 +104,13 @@ export const Minesweeper = function(appId) {
|
||||||
async function initializeLeaderBoard() {
|
async function initializeLeaderBoard() {
|
||||||
const title = `Best Times (${setting.name})`
|
const title = `Best Times (${setting.name})`
|
||||||
|
|
||||||
// remove existing leaderboard
|
|
||||||
const previousLeaderBoard = document.getElementById('leaderboard')
|
const previousLeaderBoard = document.getElementById('leaderboard')
|
||||||
const parent = previousLeaderBoard?.parentNode
|
|
||||||
if (previousLeaderBoard)
|
|
||||||
parent?.removeChild(previousLeaderBoard)
|
|
||||||
|
|
||||||
let loading = document.createElement('div')
|
let loading = document.createElement('div')
|
||||||
loadingService.addLoading(loading)
|
loadingService.addLoading(loading)
|
||||||
appElement?.append(loading)
|
if (previousLeaderBoard)
|
||||||
|
appElement?.replaceChild(loading, previousLeaderBoard)
|
||||||
|
else
|
||||||
|
appElement?.append(loading)
|
||||||
|
|
||||||
const leaderBoard = await leaderBoardService.update(setting.id ?? setting.name, title)
|
const leaderBoard = await leaderBoardService.update(setting.id ?? setting.name, title)
|
||||||
leaderBoard.id = 'leaderboard'
|
leaderBoard.id = 'leaderboard'
|
||||||
|
|
@ -124,7 +122,7 @@ export const Minesweeper = function(appId) {
|
||||||
|
|
||||||
const resetButton = document.createElement('button')
|
const resetButton = document.createElement('button')
|
||||||
resetButton.innerText = 'Reset'
|
resetButton.innerText = 'Reset'
|
||||||
resetButton.onmousedown = () => generateGrid()
|
resetButton.onmousedown = () => location.reload()
|
||||||
footBar.append(resetButton)
|
footBar.append(resetButton)
|
||||||
|
|
||||||
let levelsDropdown = document.createElement('select')
|
let levelsDropdown = document.createElement('select')
|
||||||
|
|
@ -196,8 +194,6 @@ export const Minesweeper = function(appId) {
|
||||||
|
|
||||||
|
|
||||||
function generateGrid() {
|
function generateGrid() {
|
||||||
|
|
||||||
//generate 10 by 10 grid
|
|
||||||
firstClick = true
|
firstClick = true
|
||||||
grid.innerHTML = ''
|
grid.innerHTML = ''
|
||||||
grid.oncontextmenu = () => false
|
grid.oncontextmenu = () => false
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue