Skip to content
Snippets Groups Projects

Fixes for Mobile Responsive layout - Styles.less

Merged Vijay Tilak requested to merge vijay.tilak/vtigercrm:master into master

Updated Styles.less which contains fixes for Mobile Responsive Layout

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Author Contributor

    @uma.s

    Hi Uma,

    I have updated styles.less file which contains fixes for the mobile responsive layout. Can you please check this and possibly merge it to vtiger/master?

    I have also created a new merge request for this.

    Thanks, Vijay

  • Vijay Tilak Added 1 commit:

    Added 1 commit:

    • 570784ee - Fixes for Mobile Responsive layout - updated Styles.less to remove borderbottom …
  • Author Contributor

    Hi @uma.s,

    Today morning I made another commit and I need to make one more commit to fix one more issue with the responsive layout.

    I am doing it now.

    Cheers, Vijay

  • Author Contributor

    @uma, one more doubt. Will styles.less be compiled into the respective css files when vtiger is installed?

    When I tried to install the vtiger code in my local, it didn't autocompile the vtiger.less file. So should I commit the compiled css file too?

    Cheers, Vijay

  • Vijay Tilak Added 1 commit:

    Added 1 commit:

    • 7a49da06 - Top padding for Attach File btn in comment widget
  • @vijay.tilak Less files will not be autocompiled to css files, Which has dependencies on less libraries to be installed for compilation. As you said previoulsy via your editor the less files got autocompiled, You can use the same approach for compiling less files.

  • Uma Status changed to merged

    Status changed to merged

  • Uma mentioned in commit e7971808

    mentioned in commit e7971808

  • Author Contributor

    @uma.s : Hi Uma,

    I have pushed few more fixes/refinements including the compiled css files (from less). Should I create another merge request? or will you be able to merge it again?

    Cheers, Vijay

  • @vijay.tilak You have to create a merge request and submit for review.

  • Author Contributor

    @uma.s here is the new merge request : !578 (merged)

    Thanks, Vijay

Please register or sign in to reply
Loading