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

HTML Emojis #251

Closed
wants to merge 3 commits into from
Closed

HTML Emojis #251

wants to merge 3 commits into from

Conversation

Fanreza
Copy link
Contributor

@Fanreza Fanreza commented Oct 2, 2022

Deskripsi (Description)

Checklist:

Umum:
  • Saya menambah kode basic terbaru.
  • Saya memperbaiki kode basic yang sudah ada.
  • Saya menambah template terbaru.
  • Saya memperbaiki template yang sudah ada.
  • Saya memperbaiki dokumentasi.
  • Saya menambah dokumentasi.
Contributor Requirements (Syarat Kontributor) dan Lain-Lain:
  • Saya sudah membaca (I have read) CONTRIBUTING dan sudah menyetujui semua syarat.
  • Saya telah menambahkan komentar kode yang memberikan penjelasan maksud dari kode yang saya buat.
  • Saya menggunakan bahasa Indonesia untuk memberikan penjelasan dari kode yang saya buat.

Environment

Saya menggunakan (I'm using):

  • os = windows

Testing

  • link html
  • lint css

linked issue #250

@Fanreza Fanreza requested a review from dhafitf as a code owner October 2, 2022 08:40
Comment on lines +23 to +27
| 🏿 | Dark skin tone |
| 🏾 | Medium Dark skin tone |
| 🏽 | Medium skin tone |
| 🏼 | Medium Light skin tone |
| 🏻 | Light skin tone |
Copy link
Member

Choose a reason for hiding this comment

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

Tambahkan backtick pada semua value decimal, agar tidak berubah menjadi warna seperti pada gambar ini.
image

@Fanreza Fanreza requested a review from dhafitf October 2, 2022 11:28
@Fanreza
Copy link
Contributor Author

Fanreza commented Oct 2, 2022

Perlukah saya close pull request ini lalu membuat pr baru?

@slowy07
Copy link
Member

slowy07 commented Oct 2, 2022

boleh boleh @Fanreza

@Fanreza Fanreza closed this Oct 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants