React: अतिव्यापी शैलियों का उपयोग करते समय चेतावनी दें (जैसे सीमा और सीमा रेखा)

को निर्मित 26 मार्च 2016 · 20टिप्पणियाँ · स्रोत: facebook/react

यह "अच्छा पहला बग" @YongPilMoon द्वारा लिया गया है। जब तक आप उस पर काम न करें!

इस पर कुछ समय पहले चर्चा की गई है लेकिन मुझे नहीं लगता कि कोई निष्कर्ष निकला था, और इस मुद्दे को हल करने के इच्छुक पीआरएस को विभिन्न कारणों से विलय नहीं किया गया था।

मैं उन पीआरओं को पुराने के रूप में बंद करना चाहूंगा, और इस बारे में चर्चा को फिर से शुरू करूंगा।
प्रदर्शन माना जाता है, @sebmark कचरा और @syranide से निष्कर्ष लगता है:

हमें परस्पर विरोधी शैली नियमों की अनुमति नहीं देने पर विचार करना चाहिए।

- https://github.com/facebook/react/pull/2013#issuecomment -57338177

IMHO, सभी चीजों को माना जाता है कि ओवरलैप करना और देव में चेतावनी देना बेहतर है।

- https://github.com/facebook/react/pull/4661#issuecomment -132996649

@Ianobermiller द्वारा रेडियम https://github.com/FormidableLabs/radium/issues/95 में एक ही निष्कर्ष पर आया था, लेकिन बाद में कुछ प्रतिक्रिया हुई। रिएक्ट नेटिव शैली के विस्तार की अनुमति देता है, लेकिन केवल कुछ विशेषताओं के लिए (जैसे margin और padding , लेकिन border )।

मैं इस बारे में पुराने पुल अनुरोधों को बंद कर रहा हूं, और इस मुद्दे को व्यवहार के कार्यान्वयन को ट्रैक करने के लिए बना रहा हूं जिस पर हम सहमत हैं: हमें __DEV__ में चेतावनी देना चाहिए जब border और borderBottom एक ही समय में उपयोग किया जाता है। हम इस मुद्दे में अधिक बारीकियों पर चर्चा कर सकते हैं (या तो नजरअंदाज कर दिया जाना चाहिए?

माइग्रेशन रणनीति के रूप में, हम लोगों को सुझाव दे सकते हैं कि वे https://github.com/ActionIQ/style-builder जैसे कुछ का उपयोग कर सकते हैं यदि उन्हें वास्तव में उन शॉर्टकट की आवश्यकता है। यह भी ऐसी चीज है जिसे हमें एकीकृत स्टाइल लागू करते समय तय करना होगा।

संबंधित मुद्दों:

# 2231
# 2407
# 5030
https://github.com/FormidableLabs/radium/issues/95

सबसे उपयोगी टिप्पणी

उन स्थितियों के बारे में क्या है जहां एक तृतीय-पक्ष घटक शैलियों को परिभाषित करता है और ओवरराइड की अनुमति देता है? कहते हैं कि एक घटक अपनी border शैली को परिभाषित करता है और मैं borderBottom शैली को ओवरराइड करना चाहता हूं?

aweary 29 मार्च 2016

👍8

सभी 20 टिप्पणियाँ

@gaearon इस पुनरुत्थान के लिए धन्यवाद! पागल कि मुझे रेडियम मुद्दा खोले हुए एक साल हो गया है।

मुझे border जैसे शॉर्टहैंड गुणों के बारे में चेतावनी का विचार पसंद है, लेकिन borderBottom । अधिक संपत्ति के नामों का उपयोग करते हुए शैली की वस्तुओं को मर्ज करने पर कोई आश्चर्य नहीं होना चाहिए।

सवाल। क्या इस मुद्दे की मंशा केवल व्यक्तिगत शैली की वस्तुओं के विश्लेषण को संबोधित करना है या क्या सशर्त रूप से लागू होने वाली संपत्तियों के संभावित टकरावों के बारे में विश्लेषण और चेतावनी देने की इच्छा है?

mattborn 28 मार्च 2016

सवाल। क्या इस मुद्दे की मंशा केवल व्यक्तिगत शैली की वस्तुओं के विश्लेषण को संबोधित करना है या क्या सशर्त रूप से लागू होने वाली संपत्तियों के संभावित टकरावों के बारे में विश्लेषण और चेतावनी देने की इच्छा है?

यकीन नहीं होता कि मैं इस सवाल को समझ सकता हूं, क्या आप विस्तार से बता सकते हैं?

gaearon 29 मार्च 2016

ज़रूर। यदि एक ही तत्व पर दोनों border और borderColor होना अमान्य है, तो यह इस बारे में चेतावनी देने के लिए समझ में आता है। मेरे अनुभव में, इस संयोजन से बचना आसान है, लेकिन जब शैलियों को गतिशील रूप से लागू किया जा रहा हो, तो प्रॉम्प्ट और डिबग करना अधिक कठिन होता है (प्रॉप्स या स्टेट, मीडिया क्वेश्चन, रेडियम इत्यादि के आधार पर सशर्त तर्क) या प्रॉप्स और मर्ज के रूप में पारित लोअर ऑर्डर स्टाइल ऑब्जेक्ट्स के साथ। यदि शॉर्टहैंड के लिए चेतावनी border विशेषता तब होती है जब कोई अन्य विशेषताएँ शामिल नहीं होती हैं जो सीमा को प्रभावित कर सकती हैं, तो यह चेतावनी एक सहायक डिबगिंग टूल की तुलना में एक हठी नियम की तरह लगती है क्योंकि शॉर्टहैंड border अभी भी तकनीकी रूप से मान्य है। हालाँकि, अगर एक अलग स्टाइल ऑब्जेक्ट में एक विशेषता है- जैसे borderColor - उदाहरण के लिए, इसमें शामिल सीमा प्रभावित होगी और नोड के लिए रनटाइम पर लागू होगी जो पहले से ही border शॉर्टहैंड है।

mattborn 29 मार्च 2016

आह, अच्छी बात है। मैं शायद तभी चेतावनी दूंगा जब हम _know_ शैलियों को ओवरलैप करते हैं, लेकिन मुझे यकीन नहीं है कि यह बताना कितना आसान है। मैंने इसके बारे में ज्यादा नहीं सोचा है और मेरे पास एक अच्छा समाधान नहीं है।

gaearon 29 मार्च 2016

👍2

@gaearon एक वैकल्पिक समाधान जो मुझे बेहतर लगता है वह है https://stithub.com/facebook/react/issues/5397 के अनुसार csstext का उपयोग करना, जो IIRC इन मुद्दों के एक सुपरसेट को साफ करता है।

jimfb 29 मार्च 2016

aweary 29 मार्च 2016

👍8

मैंने शॉर्टहैंड प्रॉपर्टीज को लॉन्गफॉर्म (अन्य चीजों के बीच) में बदलने के लिए डेड-सिंपल हेल्पर फंक्शन्स पर काम करना शुरू किया है: https://github.com/rtsao/lostyle/tree/master/src ( डॉक्स )

rtsao 14 सित॰ 2016

👍1

यह सुपर अप्रत्याशित और भ्रमित करने वाला है और मैं अभी इसमें भाग गया हूं।

हम सुझाव देंगे कि जब हम समस्याग्रस्त हों तो हम कम से कम अपडेट पर चेतावनी दें। यही है, अगर हम {बैकग्राउंड: ए, बैकग्राउंडाइज: बी} से {बैकग्राउंड: सी, बैकग्राउंडसाइज: बी} पर जाते हैं, तो चेतावनी दें कि बैकग्राउंडाइज को ओवरराइड किया जा रहा है (लेकिन अगर उसी समय बैकग्राउंडाइज में बदलाव होता है, तो चेतावनी न दें)। हम सभी संपत्ति आशुलिपि के लिए ऐसा कर सकते हैं और जानते हैं कि क्या यह संपत्ति गणना के आदेश से समस्याग्रस्त है।

sophiebits 24 मई 2018

👍1

"अच्छा पहला मुद्दा" के रूप में चिह्नित। इसके ऊपर मेरी टिप्पणी के लिए एक DEV- केवल चेतावनी जोड़ें। मुझे लगता है कि जाँच शुरू करने के लिए एक अच्छी जगह यहाँ होगी:

https://github.com/facebook/react/blob/v16.4.0/packages/react-dom/src/client/ReactDOMFiberComponent.js#L619

यदि आप इसे लेना चाहते हैं, तो कृपया यहाँ टिप्पणी करें ताकि हमारे पास इस पर काम करने वाले दो लोग न हों।

sophiebits 24 मई 2018

👍1

@sophiebits , मैं इस पर काम करने के लिए खुश हूँ :)

supertinou 24 मई 2018

👍1

@supertinou यह तुम्हारा है!

sophiebits 24 मई 2018

❤2

बस यह मारा और वास्तव में आश्चर्यचकित था और इसके द्वारा भ्रमित था।

craigkovatch 13 जून 2018

@supertinou बस जाँच करना चाहता था कि क्या कुछ है जो मैं इस पर आपकी मदद कर सकता हूं।

sophiebits 13 जून 2018

@sophiebits हाय मैं किसी भी पैर की उंगलियों पर कदम नहीं करना चाहता: डी, लेकिन मैंने इस मुद्दे पर एक नज़र डाली है, इसे दोहराया है और इस पर एक छुरा लेना पसंद करेगा लेकिन मुझे नहीं पता कि

mirkojotic 18 जून 2018

मैंने https://codepen.io/giuseppegurgone/pen/mjKbmq?editors=0010 @gaearon / @sophiebits पर जो कुछ चल रहा है, उसे समझाने के लिए मैंने एक कोडपेन बनाया / समझाया।