You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by corneadoug <gi...@git.apache.org> on 2015/07/06 05:20:05 UTC

[GitHub] incubator-zeppelin pull request: Fix ZEPPELIN-151

GitHub user corneadoug opened a pull request:

    https://github.com/apache/incubator-zeppelin/pull/145

    Fix ZEPPELIN-151

    Move the Font Imports to another file so it is not in the middle of minified css

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/corneadoug/incubator-zeppelin fix/fontImport

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-zeppelin/pull/145.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #145
    
----
commit a7a3c597f7b52889c0dc0a0ee15a30ef21d529af
Author: Damien Corneau <co...@gmail.com>
Date:   2015-07-06T03:18:55Z

    Fix ZEPPELIN-151

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: Fix ZEPPELIN-151

Posted by corneadoug <gi...@git.apache.org>.
Github user corneadoug closed the pull request at:

    https://github.com/apache/incubator-zeppelin/pull/145


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: Fix ZEPPELIN-151

Posted by stumped2 <gi...@git.apache.org>.
Github user stumped2 closed the pull request at:

    https://github.com/apache/incubator-zeppelin/pull/145


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: Fix ZEPPELIN-151

Posted by corneadoug <gi...@git.apache.org>.
GitHub user corneadoug reopened a pull request:

    https://github.com/apache/incubator-zeppelin/pull/145

    Fix ZEPPELIN-151

    Move the Font Imports to another file so it is not in the middle of minified css

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/corneadoug/incubator-zeppelin fix/fontImport

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-zeppelin/pull/145.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #145
    
----
commit a7a3c597f7b52889c0dc0a0ee15a30ef21d529af
Author: Damien Corneau <co...@gmail.com>
Date:   2015-07-06T03:18:55Z

    Fix ZEPPELIN-151

commit 03aae981a65666bed1659e8061afa4fb2dca4754
Author: Damien Corneau <co...@gmail.com>
Date:   2015-07-06T09:34:18Z

    Add licence in new css file

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: Fix ZEPPELIN-151

Posted by corneadoug <gi...@git.apache.org>.
Github user corneadoug commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/145#issuecomment-119199939
  
    Do you have a screenshot for me to see what is the problem?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: Fix ZEPPELIN-151

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/145#issuecomment-119204034
  
    Here's a screenshot
    ![image](https://cloud.githubusercontent.com/assets/1540981/8547250/6e3abb78-2472-11e5-90fd-a489db6a518b.png)



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: Fix ZEPPELIN-151

Posted by corneadoug <gi...@git.apache.org>.
Github user corneadoug commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/145#issuecomment-119401441
  
    Thanks for checking it. Seems like new cssmin version I set when I was trying to fix the bug originally doesn't work properly with current usemin version.
    
    I reversed the bower version of cssmin


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: Fix ZEPPELIN-151

Posted by corneadoug <gi...@git.apache.org>.
Github user corneadoug commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/145#issuecomment-120686927
  
    Did you do a Bower cache clean and delete bower_components?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: Fix ZEPPELIN-151

Posted by corneadoug <gi...@git.apache.org>.
Github user corneadoug commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/145#issuecomment-119112520
  
    CI Is Green, ready to merge


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: Fix ZEPPELIN-151

Posted by stumped2 <gi...@git.apache.org>.
Github user stumped2 commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/145#issuecomment-120987550
  
    Closing as per https://issues.apache.org/jira/browse/INFRA-9976


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: Fix ZEPPELIN-151

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/145#issuecomment-120735468
  
    After cleaning cache, i see the correct font. Thanks! i'm merging it.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: Fix ZEPPELIN-151

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/145#issuecomment-119196922
  
    Could you verify it? i tried this branch and layout looks broken


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: Fix ZEPPELIN-151

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/145#issuecomment-120654905
  
    Still, font is unchanged. Could you check again?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---