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

[TIMOB-9841] iOS: Allow to hide Network Activity Indicator #2646

Merged
merged 5 commits into from Jul 31, 2012
Merged

[TIMOB-9841] iOS: Allow to hide Network Activity Indicator #2646

merged 5 commits into from Jul 31, 2012

Conversation

mstepanov
Copy link
Contributor

TIMOB-9841

Test case in JIRA comments.


- name: disableNetworkActivityIndicator
summary: |
Disables network activity indicator from displaying.
Copy link
Contributor

Choose a reason for hiding this comment

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

Prevents network activity indicator from being displayed.

Disables network activity indicator from displaying.
description: |
Setting this property to true disables appearance of network activity indicator when a network activity is in progress.
In case network activity indicator is currently visible, it will be hidden.
Copy link
Contributor

Choose a reason for hiding this comment

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

There's two extra spaces at the beginning of this line, I think--should line up with previous line. Suggest changing it to:

If the network activity indicator is currently visible, it is hidden immediately.

@arthurevans
Copy link
Contributor

Doc reviewed, validated & ran docgen. Accepted.

@mstepanov
Copy link
Contributor Author

Hold CR until additional commit is pushed.

@arthurevans
Copy link
Contributor

Reviewed doc, ran validate and docgen. Accepted.

**NOTE: In general, the user should always be made aware of network activity. The network activity indicator should only be disabled for very brief network activity (a few seconds).**
type: Boolean
default: false
platforms: [iphone, ipad]
Copy link
Contributor

Choose a reason for hiding this comment

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

since tag is missing

@vishalduggal
Copy link
Contributor

Code Reviewed. Missing since tag otherwise ok.

@vishalduggal
Copy link
Contributor

Code Reviewed. ACCEPTED

@vishalduggal
Copy link
Contributor

Functional Passed. Ran validate and docgen. ACCEPTED

vishalduggal added a commit that referenced this pull request Jul 31, 2012
[TIMOB-9841] iOS: Allow to hide Network Activity Indicator
@vishalduggal vishalduggal merged commit e382067 into tidev:master Jul 31, 2012
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.

None yet

3 participants