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

RN 0.57.3 breaks CSS triangles on iOS #22824

Closed
lfkwtz opened this issue Dec 28, 2018 · 4 comments
Closed

RN 0.57.3 breaks CSS triangles on iOS #22824

lfkwtz opened this issue Dec 28, 2018 · 4 comments
Labels
Bug Platform: iOS iOS applications. Resolution: Locked This issue was locked by the bot.

Comments

@lfkwtz
Copy link

lfkwtz commented Dec 28, 2018

Environment

  React Native Environment Info:
    System:
      OS: macOS 10.14.2
      CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
      Memory: 191.33 MB / 16.00 GB
      Shell: 5.3 - /bin/zsh
    Binaries:
      Node: 8.12.0 - ~/.nvm/versions/node/v8.12.0/bin/node
      Yarn: 1.12.3 - ~/.yarn/bin/yarn
      npm: 6.4.1 - ~/.nvm/versions/node/v8.12.0/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
      Android SDK:
        API Levels: 28
        Build Tools: 28.0.3
        System Images: android-28 | Google APIs Intel x86 Atom
    IDEs:
      Android Studio: 3.2 AI-181.5540.7.32.5014246
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.6.3 => 16.6.3 
      react-native: 0.57.8 => 0.57.8 
    npmGlobalPackages:
      react-native-cli: 2.0.1
      react-native-git-upgrade: 0.2.7

Description

#21208 breaks CSS triangles created via borders in iOS

RN 0.57.3
simulator screen shot - iphone x - 2018-12-28 at 15 34 44

RN 0.57.2
simulator screen shot - iphone x - 2018-12-28 at 15 37 03

Reproducible Demo

        <View
          style={{
            backgroundColor: "transparent",
            borderBottomWidth: 80,
            borderBottomColor: "gray",
            borderRightWidth: 80,
            borderRightColor: "transparent",
            borderLeftWidth: 80,
            borderLeftColor: "transparent",
            width: 0,
            height: 0
          }}
        />
        <View
          style={{
            backgroundColor: "transparent",
            borderLeftWidth: 80,
            borderLeftColor: "gray",
            borderTopWidth: 80,
            borderTopColor: "transparent",
            borderBottomWidth: 80,
            borderBottomColor: "transparent",
            width: 0,
            height: 0
          }}
        />
        <View
          style={{
            backgroundColor: "transparent",
            borderTopWidth: 80,
            borderTopColor: "gray",
            borderRightWidth: 80,
            borderRightColor: "transparent",
            borderLeftWidth: 80,
            borderLeftColor: "transparent",
            width: 0,
            height: 0
          }}
        />
        <View
          style={{
            backgroundColor: "transparent",
            borderRightWidth: 80,
            borderRightColor: "gray",
            borderTopWidth: 80,
            borderTopColor: "transparent",
            borderBottomWidth: 80,
            borderBottomColor: "transparent",
            width: 0,
            height: 0
          }}
        />
@react-native-bot react-native-bot added the Platform: iOS iOS applications. label Dec 28, 2018
@simform-solutions
Copy link

https://codedaily.io/tutorials/22/The-Shapes-of-React-Native

@lfkwtz
Copy link
Author

lfkwtz commented Jan 3, 2019

That guide was probably created pre-0.57.3

screen shot 2019-01-03 at 5 22 33 pm

@fbartho
Copy link
Contributor

fbartho commented Jan 3, 2019

@lfkwtz -- Which style of triangle is correct? It looks to me like this is a regression in 0.57.3?

@lfkwtz
Copy link
Author

lfkwtz commented Feb 1, 2019

@fbartho yup it's a regression in 0.57.3 - the left style is "correct", the right style is how 0.57.3 now shows that same triangle.

@hramos hramos removed the Bug Report label Feb 6, 2019
grabbou pushed a commit that referenced this issue Apr 8, 2019
Summary:
Fixes #22824 #21945 , the bug comes from #21208 , it was to fix #11897. Now Let's constrain edge adjust only when view has corners.

[iOS] [Fixed] - Fix triangle views on iOS
Pull Request resolved: #23402

Differential Revision: D14059192

Pulled By: hramos

fbshipit-source-id: be613bf056d3cc484f281f7ea3d08f251971700a
@facebook facebook locked as resolved and limited conversation to collaborators Feb 13, 2020
@react-native-bot react-native-bot added the Resolution: Locked This issue was locked by the bot. label Feb 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug Platform: iOS iOS applications. Resolution: Locked This issue was locked by the bot.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants