Skip to content
This repository has been archived by the owner on Oct 8, 2021. It is now read-only.

overridden defaultHomeScroll config value #5751

Closed
wants to merge 1 commit into from

Conversation

peteboere
Copy link

Added 'is defined' check.

@gseguin
Copy link
Contributor

gseguin commented Mar 20, 2013

@peteboere, Thank you for the pull request! I noticed that you haven't signed a CLA. Please visit http://contribute.jquery.org/CLA/ and sign it so we can look into your PR.

Thank you.

@peteboere
Copy link
Author

Have now signed the CLA.

@arschmitz
Copy link
Contributor

@peteboere what is the use case for this? this is not a configurable default or documented in anyway?

@peteboere
Copy link
Author

At the time it was in the source as a configurable default, documented or not.

I don't know if this is still the case though, being a 5 month old pull request.

@arschmitz
Copy link
Contributor

@peteboere this is not a configurable default its a stored calculated value do you have a use case for wanting to override this?

@peteboere
Copy link
Author

This default has been exposed for a long time in jqm.

The problem with the value being non-zero is described here #2846

@jaspermdegroot
Copy link
Contributor

There is another (open) ticket for the 1px jump issue: #5488. I suggest we discuss the problem and possible solution there.

@arschmitz
Copy link
Contributor

This variable has never been documented and we intend to continue with this as it may even be removed at some point so i'm closing this.

@arschmitz arschmitz closed this Aug 8, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants