Navigation Menu

Skip to content
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

Jump to bottom

fixed an issue that display incorrect in Chrome #6831

Closed
wants to merge 1 commit into from
Closed

fixed an issue that display incorrect in Chrome #6831

wants to merge 1 commit into from

Conversation

Aqours
Copy link

@Aqours Aqours commented Dec 9, 2014

fixed an issue that display incorrect in Chrome
2014-12-09_185736

fixed an issue that display incorrect in Chrome
faab650 
fixed an issue that display incorrect in Chrome
@mnapoli
Copy link
Contributor

mnapoli commented Dec 9, 2014

I can't reproduce the issue, e.g. on http://demo.piwik.org the arrows are displayed correctly. I have Chrome 39, which version do you use?

@Aqours
Copy link
Author

Aqours commented Dec 10, 2014

Laptop(14') & Google Chrome 39.0.2171.71 (No Extension)

2014-12-10_084130
2014-12-10_084159

@mattab
Copy link
Member

mattab commented Dec 28, 2014

I think this issue maybe because the file CoreHome/angularjs/menudropdown/menudropdown.directive.less was included twice, one in Morpheus/stylesheets/base.less and also it was declared in the AssetManager.getStylesheetFiles in CoreHome

@mattab mattab self-assigned this Dec 28, 2014
@mattab mattab closed this Dec 28, 2014
@mattab mattab added the answered For when a question was asked and we referred to forum or answered it. label Dec 28, 2014
mattab added a commit that referenced this pull request Dec 28, 2014
@mattab
fixes #6831 only include file once
796c8e9
@mattab
Copy link
Member

mattab commented Dec 28, 2014

should be fixed in 796c8e9 - let me know if you still have the issue!

mattab added a commit that referenced this pull request Dec 29, 2014
@mattab
Fix regression found in screenshot test refs #6831
61c6f15
@mattab mattab added not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. and removed not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. labels Feb 16, 2015
@Aqours
Copy link
Author

Aqours commented Feb 16, 2015

I still have this problem.

Google Chrome: 40.0.2214.111 (Stable) m
Piwik: Version 2.10.0

@Aqours
Copy link
Author

Aqours commented Feb 16, 2015

The blue triangle should be control of the code（CSS）

.menuDropdown .title:after {
    ...; 
    border-left: 4px solid transparent;
    border-right: 4px solid transparent;
    border-top: 5px solid #4183C4;
    ...;
}

I don't understand the function of this code（CSS）①

.menuDropdown .title:after { 
   ...;
   content: " \25BC";
   ...;
}

① Can display in Chrome, but don't display in Firefox or IE. And I use simple code to test that.
The code

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>mytest</title>
    <style>
        .t:after {
          content: '';
          top: 5px;
          right: -15px;
          color: #4183C4;
          display: inline;
          content: " \25BC";
          font-size: 1px;
          height: 0px;
          width: 0px;
          border-left: 4px solid transparent;
          border-right: 4px solid transparent;
          border-top: 5px solid #4183C4;
        }
    </style>
</head>
<body>
    <div class="t">mytest</div>
</body>
</html>

The screenshot of different browser displayed.
Chrome ↓↓↓
2015-02-16_220539

Firefox ↓↓↓

2015-02-16_220600

It will display correctly in Chrome when I delete the code

.t:after { 
   content: " \25BC";
}

@mattab
Copy link
Member

mattab commented Feb 16, 2015

@MinonHeart which version are you using? can you try with 2.11.0-RC if you are not using this yet? http://forum.piwik.org/read.php?2,124222,124222#msg-124222

@Aqours
Copy link
Author

Aqours commented Feb 17, 2015

@mattab I'm using piwik-2.10.0.
@mnapoli The problem still exist when I try piwik-2.11.0-RC or visit http://demo.piwik.org on another PC.

@mattab
Copy link
Member

mattab commented Feb 17, 2015

@MinonHeart Can you please create an issue with the screenshot and browser used ? thanks

@Aqours Aqours mentioned this pull request Feb 17, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mattab mattab

Labels
answered For when a question was asked and we referred to forum or answered it. not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Aqours @mnapoli @mattab