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

Vertical Option in Family Tree View #119

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rojsiroj
Copy link

Deskripsi

Dalam PR ini, saya membuat fungsi untuk melihat pohon keluarga secara vertikal, yang sebelumnya hanya dapat dilihat secara horizontal.

Untuk dukungan CSS, saya ambil dari sumber ini Codepen dengan sedikit modifikasi agar lebih sesuai.

Checklist

  • Menambahkan controller untuk view vertical di UsersController.php
  • Membuat css untuk view vertical tree-vertical.css
  • Menambahkan translasi untuk kata 'vertikal' dan 'horisontal' dalam tiga bahasa yang disupport
  • Membuat layout family-tree.blade.php untuk sebagai layout utama pohon keluarga
  • Membuat view tree-vertical.blade.php sebagai view untuk menampilkan pohon keluarga secara vertical

Screenshot

Pohon Keluarga Vertikal

@nafiesl
Copy link
Owner

nafiesl commented Apr 17, 2024

Halo mas @rojsiroj terima kasih utk PR nya mas. Ijin mohon waktu utk review ya.

Btw kita ada grup komunitas di telegram utk diskusi project silsilah ini. Silakan gabung mas:
https://t.me/silsilah_id

Terima kasih

@rojsiroj
Copy link
Author

Baik mas @nafiesl, terimakasih

Kebetulan saya butuh buat catet silsilah keluarga, dan butuh buat view secara vertikal nya jadi sekalian aja dibuat PR hehehe

Projeknya sangat bermanfaat mas, mantap

@rojsiroj rojsiroj changed the title feat: add vertical tree option in family tree Vertical Option in Family Tree View Apr 18, 2024
@rojsiroj
Copy link
Author

rojsiroj commented Jun 1, 2024

Halo mas @nafiesl untuk PR ini gimana kelanjutan nya ya? Terima kasih

@nafiesl nafiesl self-requested a review October 30, 2024 23:25
Copy link
Owner

@nafiesl nafiesl left a comment

Choose a reason for hiding this comment

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

Halo mas @rojsiroj, mohon maaf saya baru respons mas. untuk PR ini saya ada masukan terkait kode nya (code review di bawah.)

Kemudian selain dari itu, saya mencoba nya pada pohon keluarga saya sendiri, tetapi hasilnya belum bisa rapi (screenshot terlampir).

Apakah bisa dibuat tampilannya agar seluruh anak selalu berposisi di bawah ayah/ibunya mas? Karena kondisi saat ini, seseorang yang punya banyak keturunan pohon keluarga verticalnya akan berantakan seperti gambar di bawah ini.

Terima kasih mas.

04ImFLa17V

@@ -47,7 +47,8 @@
Route::get('users/{user}/edit', 'edit')->name('users.edit');
Route::patch('users/{user}', 'update')->name('users.update');
Route::get('users/{user}/chart', 'chart')->name('users.chart');
Route::get('users/{user}/tree', 'tree')->name('users.tree');
Route::get('users/{user}/tree/vertical', 'treeVertical')->name('users.tree-vertical');
Copy link
Owner

Choose a reason for hiding this comment

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

Halo mas, di sini kita tetap perlu default action untuk endpoint /tree

Suggested change
Route::get('users/{user}/tree/vertical', 'treeVertical')->name('users.tree-vertical');
Route::get('users/{user}/tree', 'treeVertical')->name('users.tree');
Route::get('users/{user}/tree/vertical', 'treeVertical')->name('users.tree-vertical');

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.

2 participants