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
Remove scrolling conditional and allow auto-detect board when no board selected #2554
Conversation
855b1f1
to
ce8e1b1
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
src/js/tabs/firmware_flasher.js
Outdated
@@ -29,6 +29,7 @@ firmware_flasher.initialize = function (callback) { | |||
self.parsed_hex = undefined; | |||
|
|||
const unifiedSource = 'https://api.github.com/repos/betaflight/unified-targets/contents/configs/default'; | |||
const divReleaseInfo = 'div.release_info'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the element $('div.release_info)
exist at this point?
If yes, usually we create directly the element:
const divRealeaseInfoElement = $('div.release_info');
and we use it later in the rest of the code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found out something weird. Using the $(selector) as a constant was not evaluated at the places we are using them.
I mean when using $('selector') it works not when it's assigned to a constant 😕
Im not sure about removing our current autoscroll here @haslinghuis , for me is a good functionality, maybe we can go to a nonscroll fix in some cases? |
ce8e1b1
to
c622d39
Compare
c622d39
to
c138fb5
Compare
c138fb5
to
03aeb41
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
@asizon made scrolling conditional |
AUTOMERGE: (PASS)
|
Fixes: #2551
Fixes: #2609