-
Notifications
You must be signed in to change notification settings - Fork 62
MaxSize for attachments & UI #33
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
base: master
Are you sure you want to change the base?
Conversation
Added the two introductory pages, “Introduction” & “Get Started” and their controllers/design. Fixed the flow to redirect to the first page when setting up, and from them to the Welcome page.
Added the case in which the user is Logged in but does not have authorizations. Fixed the flow accordingly. Fixed the design of the rest.
…' of https://github.com/KamilaHasanbega/e2email into show-attachments
…/e2email into show-attachments
|
There is no CSS fix in this branch (but it's ok, just rename the PR to mention displaying the size of the attachments in a better way). |
chrome/pages/messages/messages.html
Outdated
| <img class="fade-show email-image-content" ng-src="{{mime.content}}" ng-if="mime.type=='image'"> | ||
| <div class="email-text-content col-xs-12" ng-if="mime.type=='text'">{{mime.content}}</div> | ||
| <div class="col-xs-12 single-attachment-received single-attachment" ng-if="mime.url"><span class="file-description"><a blob-href="{{mime.url}}" download="{{mime.filename}}">{{ mime.filename | limitTo: 28 }}{{mime.filename.length > 28 ? '...' : ''}}</a></span><span class="file-size">{{'(' + mime.filesize + ')'}}</span></div> | ||
| <div class="col-xs-12 single-attachment-received single-attachment" ng-if="mime.url"><span class="file-description"><a blob-href="{{mime.url}}" download="{{mime.filename}}">{{ mime.filename | limitTo: 28 }}{{mime.filename.length > 28 ? '...' : ''}}</a></span><span class="file-size" ng-bind="messagesCtrl.convertBytes(mime.filesize)"> {{}} </span></div> |
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.
You don't need {{}}. ng-bind should be enough.
| <div class="row"> | ||
| <div class="col-xs-12 text-center"> | ||
| <button ng-disabled="authorizationCtrl.inProgress" class="btn btn-primary btn-focus" data-ng-click="authorizationCtrl.signIn()">{{'openGmailSettings' | translate}}</button> | ||
| <button ng-disabled="authorizationCtrl.inProgress" class="btn btn-primary btn-focus centreBtn" data-ng-click="authorizationCtrl.signIn()">{{'openGmailSettings' | translate}}</button> |
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.
centreBtn => centerBtn
-Fixed size of attachments by the library.
-Fixed other CSS