Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Solve a bug in Disqus #945

Merged
merged 1 commit into from
Mar 22, 2023
Merged

Solve a bug in Disqus #945

merged 1 commit into from
Mar 22, 2023

Conversation

v4vong
Copy link
Contributor

@v4vong v4vong commented Mar 21, 2023

Description

Disqus can't be loaded due to an uncaught syntax error: Identifier 'modeToggle' has already been declared at elsewhere. The solution is to use another name.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

How has this been tested

  • I have run bash ./tools/test (at the root of the project) locally and passed
  • I have tested this feature in the browser

Test Configuration

  • Browser type & version: Chrome
  • Operating system: MacOS
  • Ruby version:
  • Bundler version:
  • Jekyll version:

Copy link
Owner

@cotes2020 cotes2020 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed, a variable with the same name has already been declared in the <script> of the head.

@v4vong
Copy link
Contributor Author

v4vong commented Mar 22, 2023

Suggestions accepted.

@cotes2020 cotes2020 merged commit 6fec411 into cotes2020:master Mar 22, 2023
nabin-nath added a commit to nabin-nath/nabin-nath.github.io that referenced this pull request Mar 29, 2023
* chore(deps): upgrade NPM packages

* chore: update code style config

* refactor(build): modularize JS code

- replace gulp with rollup
- remove JS output from repo

* chore(ci,tools): adapt to changes in JS builds

* chore: update configure-pages action (cotes2020#931)

* feat: change TOC plugin to `tocbot` (cotes2020#774)

* style: improve code style of SCSS & YAML

* fix: notch status bar doesn't match theme color (cotes2020#918)

Resolves cotes2020#918

* refactor: remove version number from self host config

* fix: some console snippets will be incompletely copied

Sample:

```console
echo $PATH
```

* style: fix scss lint issue

type: declaration-block-no-redundant-longhand-properties

* feat(ux): turn home page posts into clickable cards (cotes2020#895)

Resolves cotes2020#895

* chore: recover accidentally deleted LQIP style

* chore(tools): resume git checkout in initialization

* chore(tools): add JS dist to gem release

* docs: migrate upgrade details to the wiki

* chore(release): 5.6.0

* wip

* fix: mode-toggle leads to Disqus loading failure (cotes2020#945)

* search fix

* wip

---------

Co-authored-by: Cotes Chung <[email protected]>
Co-authored-by: Josh Johanning <[email protected]>
Co-authored-by: Song-Zi Vong <[email protected]>
HOKAGO-MEMORIES added a commit to HOKAGO-MEMORIES/hokago-memories.github.io that referenced this pull request Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants