Skip to content

Fix mobile responsive#682

Open
Sarim2022 wants to merge 2 commits intorailsgirls:mainfrom
Sarim2022:fix-mobile-responsive
Open

Fix mobile responsive#682
Sarim2022 wants to merge 2 commits intorailsgirls:mainfrom
Sarim2022:fix-mobile-responsive

Conversation

@Sarim2022
Copy link
Copy Markdown

fix issue Mobile view #542

Comment thread js/guides.js

function toggleTheme() {
const currentTheme = document.documentElement.getAttribute('data-theme');
const newTheme = currentTheme === 'dark' ? 'light' : 'dark';
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).

Comment thread js/guides.js
const toggleBtnMobile = $('#theme-toggle-mobile');

function toggleTheme() {
const currentTheme = document.documentElement.getAttribute('data-theme');
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).

Comment thread js/guides.js

function initializeThemeToggle() {
const toggleBtn = $('#theme-toggle');
const toggleBtnMobile = $('#theme-toggle-mobile');
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).

Comment thread js/guides.js
}

function initializeThemeToggle() {
const toggleBtn = $('#theme-toggle');
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).

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