na-arabox:0.1.0 and ornacover:0.1.0#5353
Conversation
|
"Hello, thank you for your time. I have submitted the na-arabox package (version 0.1.0), and all automated checks have passed successfully. Could you please review it when you have a moment? I am looking forward to contributing this to the community." |
1 similar comment
|
"Hello, thank you for your time. I have submitted the na-arabox package (version 0.1.0), and all automated checks have passed successfully. Could you please review it when you have a moment? I am looking forward to contributing this to the community." |
elegaanz
left a comment
There was a problem hiding this comment.
Thank you for this package. Just a small remark regarding the package description. Could you also fill the provided PR template? In particular, I would like to understand where the name comes from to make sure it fits our guidelines (even if it seems to be fine from what I can tell).
| entrypoint = "lib.typ" | ||
| authors = ["Teacher Nassim"] | ||
| license = "MIT" | ||
| description = "حزمة لإنشاء صناديق عربية تدعم RTL." |
There was a problem hiding this comment.
Can you please add an English version of the description? It will help people who don't speak Arabic to understand what the package is about, even if they are not the primary audience.
| @@ -0,0 +1,12 @@ | |||
| # na-arabox | |||
|
|
|||
| حزمة لـ Typst تتيح إنشاء صناديق مخصصة تدعم الكتابة باللغة العربية (RTL). | |||
There was a problem hiding this comment.
Same here, a short description in English would be appreciated.
|
Hello, Thank you for your review. I have updated the typst.toml and README.md files to include the required English descriptions, and I have also added an optional shadow feature to the box. Regarding the package name 'na-arabox', it stands for 'Naam Arabic Box' (Naam being my first name). Thank you for your time and assistance. |
| [template] | ||
| path = "lib.typ" |
There was a problem hiding this comment.
This section should be removed, since you don't actually have a template. More info here.
|
"Hello Typst team, I have submitted my package na-arabox for review and inclusion in the Typst Universe. This package provides support for Arabic text processing and right-to-left layout configurations, which I have developed to facilitate educational material design. I would appreciate it if you could review my submission. Please let me know if there are any adjustments or improvements required. Thank you for your time and effort." |
|
"Hello, Thank you for the feedback. I have updated the submission as requested: Removed the [template] section from typst.toml. Added a short English description to the README.md file. Regarding the name na-arabox, it stands for 'Nassim's Arabic Box', reflecting its purpose as a tool for creating customized Arabic layout boxes for educational content. Please let me know if any further adjustments are needed. Thank you!" |
|
"Hello,
Thank you for the feedback. I have updated the submission as requested:
Removed the [template] section from typst.toml.
Added a short English description to the README.md file.
Regarding the name na-arabox, it stands for 'Nassim's Arabic Box',
reflecting its purpose as a tool for creating customized Arabic layout
boxes for educational content.
Please let me know if any further adjustments are needed. Thank you!"
Le ven. 17 juil. 2026 à 09:23, Ana Gelez ***@***.***> a
écrit :
… ***@***.**** commented on this pull request.
------------------------------
In packages/preview/na-arabox/0.1.0/typst.toml
<#5353 (comment)>:
> +[template]
+path = "lib.typ"
This section should be removed, since you don't actually have a template. More
info here
<https://github.com/typst/packages/blob/main/docs/manifest.md#templates>.
—
Reply to this email directly, view it on GitHub
<#5353?email_source=notifications&email_token=CDUKILOWXQ4SCD6OOHWZEZD5FHO6VA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTINZSGA3TMMJXG4Y2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KYZTPN52GK4S7MNWGSY3L#pullrequestreview-4720761771>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/CDUKILOQFTM7QVRCPCE2STD5FHO6VAVCNFSNUABFKJSXA33TNF2G64TZHM3DKOBXGUZTMMRUHNEXG43VMU5TIOBYGU3TQMRSHEYKC5QC>
.
Triage notifications, keep track of coding agent tasks and review pull
requests on the go with GitHub Mobile for iOS
<https://github.com/notifications/mobile/ios/CDUKILJBLXLUYXRF5HGROGL5FHO6VA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTINZSGA3TMMJXG4Y2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KUZTPN52GK4S7NFXXG>
and Android
<https://github.com/notifications/mobile/android/CDUKILOTPPHHMPWZKF7GXLL5FHO6VA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTINZSGA3TMMJXG4Y2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2K4ZTPN52GK4S7MFXGI4TPNFSA>.
Download it today!
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
"Hello Typst team,
I have submitted my package na-arabox for review and inclusion in the Typst
Universe. This package provides support for Arabic text processing and
right-to-left layout configurations, which I have developed to facilitate
educational material design.
I would appreciate it if you could review my submission. Please let me know
if there are any adjustments or improvements required.
Thank you for your time and effort."
Le dim. 19 juil. 2026 à 10:22, nasim ahmed ***@***.***> a écrit :
… "Hello,
Thank you for the feedback. I have updated the submission as requested:
Removed the [template] section from typst.toml.
Added a short English description to the README.md file.
Regarding the name na-arabox, it stands for 'Nassim's Arabic Box',
reflecting its purpose as a tool for creating customized Arabic layout
boxes for educational content.
Please let me know if any further adjustments are needed. Thank you!"
Le ven. 17 juil. 2026 à 09:23, Ana Gelez ***@***.***> a
écrit :
> ***@***.**** commented on this pull request.
> ------------------------------
>
> In packages/preview/na-arabox/0.1.0/typst.toml
> <#5353 (comment)>:
>
> > +[template]
> +path = "lib.typ"
>
> This section should be removed, since you don't actually have a template. More
> info here
> <https://github.com/typst/packages/blob/main/docs/manifest.md#templates>.
>
> —
> Reply to this email directly, view it on GitHub
> <#5353?email_source=notifications&email_token=CDUKILOWXQ4SCD6OOHWZEZD5FHO6VA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTINZSGA3TMMJXG4Y2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KYZTPN52GK4S7MNWGSY3L#pullrequestreview-4720761771>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/CDUKILOQFTM7QVRCPCE2STD5FHO6VAVCNFSNUABFKJSXA33TNF2G64TZHM3DKOBXGUZTMMRUHNEXG43VMU5TIOBYGU3TQMRSHEYKC5QC>
> .
> Triage notifications, keep track of coding agent tasks and review pull
> requests on the go with GitHub Mobile for iOS
> <https://github.com/notifications/mobile/ios/CDUKILJBLXLUYXRF5HGROGL5FHO6VA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTINZSGA3TMMJXG4Y2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KUZTPN52GK4S7NFXXG>
> and Android
> <https://github.com/notifications/mobile/android/CDUKILOTPPHHMPWZKF7GXLL5FHO6VA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTINZSGA3TMMJXG4Y2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2K4ZTPN52GK4S7MFXGI4TPNFSA>.
> Download it today!
> You are receiving this because you authored the thread.Message ID:
> ***@***.***>
>
|
na-arabox:0.1.0
هذه حزمة لإنشاء صناديق مخصصة تدعم الكتابة باللغة العربية (RTL) في Typst.