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

Fix medial yeh #25

Merged
merged 7 commits into from
Jan 18, 2024
Merged

Conversation

amerharb
Copy link
Contributor

@amerharb amerharb commented Jan 2, 2024

وصف الفكرة (Feature Idea)

بعض الحروف ‏مثل حرف ن ي ق ‏تتعرض للتشويه في بعض الخطوط او محررات النصوص التي لا تدعم الحروف بدون تنقيط. ‏كما هي الحال في الكود الحالي كيف تمت معالجة حالة حرف النون ‏استبدالها بحرف الباء بدون تنقيط. ‏الفكرة أن يتم اضافة اوبشنز ‏اختياري أثناء التحويل ‏حتى تتم عملية التحويل هذه ‏بشكل اختياري فيكون المستهلك للمكتبة هو من يحدد طريقة التحويل

الهدف (Goal)

‏تحويل الياء من ‏هذا الشكل
image
‏الى ‏هذا الشكل:
image

توصيف تفصيلي (Detailed Description)

‏تم اضافة خياران لحرفي النون والياء

const defaultOldArabicOptions: OldArabicOptions = {
	replaceMidNoonWithBah: true,
	replaceMidYahWithBah: false,

};

‏بالنسبة لحرف النون فإن الخيار مفعل بشكل افتراضي باعتبار ان ‏ ‏معظم ‏محررات النصوص تظهر ن منقطة

الأهمية (Priority)

  • عالية الأهمية
  • متوسطة الأهمية
  • منخفضة الأهمية

@AbdelrahmanBayoumi AbdelrahmanBayoumi added enhancement New feature or request Scripts Text manipulation labels Jan 18, 2024
@AbdelrahmanBayoumi
Copy link
Member

@amerharb

جزاك الله خيراً على مجهودك، لا أدري لماذا لم يصل إليَّ إشعار بخصوص هذا الـ PR
سأراجعه الآن 🚀

Copy link
Member

@AbdelrahmanBayoumi AbdelrahmanBayoumi left a comment

Choose a reason for hiding this comment

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

جزاك الله خيراً على هذه المساهمة القيمة! التغييرات المقترحة في التعامل مع مشكلة حرف الياء بدون نقط لم ألاحظها.. وفكرة جيده أيضاً إضافة الخيارات الاختيارية للمستخدمين ليتمكنوا من تحديد طريقة التحويل بأنفسهم. أوافق على دمج هذا التغيير وأتطلع لرؤيته مطبقاً في الإصدار القادم من المكتبة إن شاء الله

@AbdelrahmanBayoumi AbdelrahmanBayoumi merged commit addf8c4 into Seen-Arabic:main Jan 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Scripts Text manipulation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants