zcuric
26 يونيو 2020، 8:00ص
1
لقد قمت بإنشاء طلب دمج (PR) لإضافة خيار لتغيير فاصل العنوان:
master ← zcuric:feature/topic-title-separator
closed 08:03AM - 29 Jun 20 UTC
### This PR
- [x] Adds `document_title_separator` setting to update the title s… eparator to enable different types of SEO titles, for example: `Test Topic | Discourse`. Defaults to `" - "`, as it was before.
- [x] Adds locale option for this new option
Setting is an enum with list of separators inspired by [Yoast SEO WP plugin](https://yoast.com/wordpress/plugins/seo/).
#### Note about tests
I've tried to write a meaningful test for this new option, but because the `qunit` creates it's own titles I coudn't verify in the test that this option applies. `qunit` titles are separated with commas, for example: `Test Topic,Qunit Discourse Test`.
الإعداد عبارة عن قيمة من نوع تعداد (enum) تحتوي على قائمة بفاصل مستوحاة من إضافة Yoast SEO لنظام ووردبريس .
على سبيل المثال، يمكنك تعيين الفاصل ليكون على النحو التالي:Discourse Meta | The Official Support Forum for DiscourseDiscourse Meta < The Official Support Forum for Discourse
حاليًا، الخيار الوحيد هو -.
لم أختار اتباع مسار الإضافة (plugin) لعدة أسباب، والسبب الرئيسي هو أنه يبدو لي أن الإضافة لا تستطيع تعديل كل هذا. قد أكون مخطئًا.
@sam @codinghorror
إعجاب واحد (1)
Stephen
26 يونيو 2020، 9:09ص
2
لست متأكدًا حقًا من أن هذا الأمر منطقي ليكون جزءًا من النواة.
إنه ليس إعدادًا لـ “المستخدم الخبير” في تحسين محركات البحث (SEO) بالمعنى الحرفي، بل هو مجرد مسألة تفضيل شخصي.
إعجابَين (2)
sam
29 يونيو 2020، 12:53ص
4
نعم، رأيي في هذا هو أنه يمكن تنظيف النواة إلى حد ما لتصبح قابلة للتوسع عبر إضافة، لكنني لا أريد حقًا حمل هذا الخيار الإضافي في النواة.
إعجابَين (2)
zcuric
29 يونيو 2020، 8:03ص
5
إغلاق طلب الدمج. شكرًا على الملاحظات.
إعجابَين (2)