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
refactor(android)(8_1_X): ProgressIndicator dialog handling #11142
Conversation
- [TIMOB-27104] ProgressIndicator logs "WindowLeaked" exception when hiding dialog and closing window at same time as of 8.0.2 - [TIMOB-27308] Determinant ProgressIndicator ignores "value" property before shown - [TIMOB-27309] Cannot re-show ProgressIndicator dialog if auto-closed by previous window - Removed dialog's handleMessage() related code. No longer needed since JS runs on main UI thread.
* Please see the LICENSE included with this distribution for details. | ||
*/ | ||
/* eslint-env mocha */ | ||
/* global Titanium */ |
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.
⚠️ tests/Resources/ti.ui.android.progressindicator.addontest.js line 8 – 'Titanium' is defined but never used. (no-unused-vars)
|
int thePos = (value - min) * incrementFactor; | ||
|
||
handler.obtainMessage(MSG_PROGRESS, thePos, -1).sendToTarget(); | ||
int progressValue = (TiConvert.toInt(newValue) - this.min) * this.incrementFactor; |
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.
Set default to zero?
TiConvert.toInt(newValue, 0)
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.
Yeah, I'll do that. Especially since we're doing this when opening the dialog too.
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.
CR: PASS
FR Passed. The progress indicator works as expected. |
* 8_1_X: chore(release): bump version chore(8_1_X): update aps-analytics (tidev#11179) fix(ios): console and tiapi multiple parameter log (tidev#11162) fix(ios): use device build as base for universal module framework (tidev#11120) fix(ios)(8_1_X): properly load watchsession module (tidev#11174) fix(ios): remove invalid architecture from framework for adhoc build (tidev#11087) refactor(android)(8_1_X): ProgressIndicator dialog handling (tidev#11142) refactor(android): consolidate incremental build fixes back into task (tidev#10997)
JIRA:
Cherry-pick of PR: #11115