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

translate render and commit #434

Merged
merged 2 commits into from May 7, 2023

Conversation

serkanisyapan
Copy link
Collaborator

@serkanisyapan serkanisyapan commented May 5, 2023

Commit'i React'in yapılan değişiklikleri render etme tamamlandıktan sonra DOM'a uygulaması, DOM'u değiştirmesi gibi çevirdim. Sizin de düşünceleriniz bekliyorum.

@github-actions
Copy link

github-actions bot commented May 5, 2023

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@baspinarenes
Copy link
Contributor

Commit'i React'in yapılan değişiklikleri render etme tamamlandıktan sonra DOM'a uygulaması, DOM'u değiştirmesi gibi çevirdim. Sizin de düşünceleriniz bekliyorum.

Virtual DOM'daki değişikliklerin DOM'a uygulanma aşaması olarak düşünüldüğünde "commit" kelimesini DOM'u değiştirmek olarak çevirmek anlam kaybına neden olabilir. State ve context'i birebir kullanıyoruz belki bu kelimeyi de kullanabiliriz. @alioguzhan abi nasıl ilerleyelim?

@alioguzhan
Copy link
Member

Disaridayim 2-3 saat sonra bakacagim buraya. Ben de bi incelemek istiyorum cunku. @serkanisyapan @baspinarenes

@alioguzhan
Copy link
Member

Bence cevirebiliriz. Cunku bana ingilizcesi bile ilk basta bi'sey ifade etmedi. icerigi okumaya baslayinca anlam kazandi.


Masallah commit kelimesinin bilmedigim 20 tane daha anlami varmis. Biraz incelemek durumunda kaldim. Simdi soyle:

Bizim contextimize en uygun muhtemel commit cevirileri:

  • işlemek
  • teslim etmek
  • sevk etmek

bunların dışındaki diğer anlamları komple eliyorum.

Ilk basta teslim etmek cok mantikli gelmisti. Cunku verdigi analojide de, commiting fazini siparisi masaya servis etmek kismina benzetmis. Bizim contextte de render edilmis bileseni DOM'a teslim etmek / yerlestirmek gibi dusunebiliriz. Ki gercekten de boyle.

Sonra Commiting to the DOM olayi var. Aslinda orda dedigi sey Committing components to the DOM.

Bu sekilde ele alip degerlendirince de DOM'a işlemek (ya da DOM'a uygulamak) daha mantikli bir ceviri oluyor. (Bilesenleri DOM'a islemek)

Sonuc olarak sayfadaki ceviriler icin konusacak olursak:

  1. Render and Commit -> Render et ve İşle
  2. Committing to the DOM -> DOM'a işlemek
  3. React commits changes to the DOM -> React degisiklikleri DOM'a işler

Benim fikrim bu sekilde. Bunun uzerine konusabiliriz. Fikirlere acigim. @baspinarenes @serkanisyapan .

@agmmnn nin cevirileri de guzeldi. Onun da fikrini alalim uygunsa.

@agmmnn
Copy link
Contributor

agmmnn commented May 6, 2023

<Profiler>'ı çevirirken commit'i aynı şekilde bırakmıştım ben, sarmaladığı komponent her güncellendiğinde callback'e değişikliği iletiliyordu, buradaki anlamı teslim etmek gibi bildiğimiz git commit'i canlandırdığı için bırakmıştım ama yine de çevrilse de pek anlam erozyonu olmazmış sanki.

Bu konuya gelirsek kontekst farklı kullanıldığı anlam da farklı, burada deliver etmek anlamından ziyade değişikliği DOM'da yerine getirme/gerçekleştirme/gerçeğe geçirme işlemini işaret ediyor haliyle DOM'a işlemek gayet yerinde çeviri olmuş, biraz düşündüm ama benim de aklıma daha iyi bir seçenek gelmedi. Değiştirmek uymamış, zaten konu içinde DOM'u değiştirmek de geçiyor (React only changes the DOM... React, DOM'u değiştirir...) hem change hem commit değiştirmek olmuş.

Thesaurus'a göre: (@alioguzhan'ın da dediği gibi)
perform an action: işlemek, gerçekleştirmek, yapmak
deliver, entrust: teslim etmek, sevk etmek, devretmek

chrome_ZdVxgfNFyb

@serkanisyapan
Copy link
Collaborator Author

Teşekkürler herkese fikirler için. DOM'a işlemek olarak çeviriyi düzelttim. Kelimeyi ortak çevirilere ekleyebiliriz sanırım birkaç yerde daha kullanılıyor. @alioguzhan @baspinarenes

Copy link
Member

@alioguzhan alioguzhan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tesekkurler

@alioguzhan alioguzhan merged commit 9e6ac42 into reactjs:main May 7, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants