bp-default - media screen css error when minifying

[danbp]

Possible css error on media screen.

See related post for more details
​https://buddypress.org/support/topic/issue-with-w3-total-cache/

[r-a-y]

The problem referenced in the thread is with the bp-default theme and not bp-legacy:
​https://buddypress.trac.wordpress.org/browser/tags/2.0.3/bp-themes/bp-default/_inc/css/responsive.css#L121

I don't know if we should open up the "Issues" tab on the Github bp-default repo for things like this.

[boonebgorges]

What's the correct syntax?

I don't know if we should open up the "Issues" tab on the Github bp-default repo for things like this.

Seems like it might be easier to keep issues here until the status of bp-default has changed a bit more - like when other people are committing to it. Happy to hear other opinions though.

[r-a-y]

What's the correct syntax?

We need to add a selector when using the ​-webkit-text-size-adjust rule:

Here's a potential fix:

@media screen and (max-device-width: 480px), screen and (-webkit-min-device-pixel-ratio: 2) {
   #buddypress {
        -webkit-text-size-adjust: none;
   } 
}

However, I'm not too familiar with this rule, so need some feedback here.

---

Seems like it might be easier to keep issues here until the status of bp-default has changed a bit more - like when other people are committing to it.

I'm okay with this.

[DJPaul]

If someone wants to test and patch this and send a pull request on Github, we're happy to merge it in.

[DJPaul]

Closing most tickets related to BP-Default and BP-Legacy, since the upcoming BP-Nouveau template pack (planned for 3.0) will make these redundant.

[DJPaul]

Closing most tickets related to BP-Default and BP-Legacy, since the upcoming BP-Nouveau template pack (planned for 3.0) will make these redundant.