-
Notifications
You must be signed in to change notification settings - Fork 46
display no documents and loading message #109
base: master
Are you sure you want to change the base?
Conversation
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.
Thanks a lot for your pull requests!
I have some remarks, could you take a look at it?
return Container( | ||
height : 100, | ||
margin : EdgeInsets.symmetric(horizontal : 100, vertical : 100), | ||
child : Column( |
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.
I think a CircularProgressIndicator
would look better instead of the text.
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.
Also, this may be shown if the request fails. Can you change this so it only shows when requesting == true
?
style : TextStyle(fontSize : 15), | ||
textAlign : TextAlign.center,), | ||
|
||
Text(('$searchString').i18n, |
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.
Please remove the .i18n
, we don't want to translate the search string 🙂
margin : EdgeInsets.symmetric(horizontal : 100, vertical : 100), | ||
child : Column( | ||
children:[ | ||
Text("There are no documents \n containing".i18n, |
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.
Any reason why you didn't want to use Heading
instead of Text
? Are there visual issues?
I fix #108 (comment).
(add no document message and loading message)