-
Notifications
You must be signed in to change notification settings - Fork 163
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
[UX] servers page : make servers page to be responsive on mobile #1005
[UX] servers page : make servers page to be responsive on mobile #1005
Conversation
modules/imap/output_modules.php
Outdated
|
||
// IMAP Username | ||
$res .= '<div class="form-floating">'; | ||
$res .= '<input '.$disabled.' id="imap_user_'.$server_id.'" class="form-control credentials" type="text" name="imap_user" value="'.$this->html_safe($user_pc).'" placeholder="'.$this->trans('Username').'">'; | ||
$res .= '<input '.$disabled.' id="imap_user_'.$server_id.'" class="form-control " type="text" name="imap_user" value="'.$this->html_safe($user_pc).'" placeholder="'.$this->trans('Username').'">'; |
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.
@Bienvenumoringa you should not remove class credentials as it is used in javascript. Just remove its css.
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've already put back the credentials class so that the js works as expected. And I've removed it from the css side.
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.
@Bienvenumoringa this fix only for IMAP servers but not SMTP servers
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.
Applied changes
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.
@Bienvenumoringa there is too much space between 2 servers. Can you fix ?
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.
Applied changes
19726a5
to
921a460
Compare
921a460
to
9638e97
Compare
9638e97
to
6d9398e
Compare
…pht-org#1005) * [UX] servers page : make servers page to be responsive on mobile * Arranged mobile servers to look good --------- Co-authored-by: Josaphat Imani <[email protected]>
Working on responsive design of servers page
?page=servers
, see screenshot :