Feat: hapus informasi field inputan pada detail penduduk#1663
Open
pandigresik wants to merge 3 commits into
Open
Feat: hapus informasi field inputan pada detail penduduk#1663pandigresik wants to merge 3 commits into
pandigresik wants to merge 3 commits into
Conversation
Contributor
|
🔄 AI PR Review sedang antri di server...
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request: Feat: Tampilkan Detail Data Penduduk Tanpa Field Inputan
Description
Mengubah tampilan detail data penduduk (view details) agar hanya menampilkan data dalam bentuk statis (
form-control-static) tanpa menggunakan elemen field inputan yang di-disable. Perubahan ini menyelesaikan masalah estetika dan fungsionalitas di mana data yang hanya dibaca (read-only) sebelumnya masih menggunakan input textbox, dropdown, dan radio button yang dinonaktifkan.Changes made:
<input type="text" ... disabled>) menjadi elemen paragraf statis (<p class="form-control-static">) pada fileresources/views/data/penduduk/form_show.blade.php.<select disabled>) dan radio button group untuk Jenis Kelamin, Status Penduduk, e-KTP, Agama, Pendidikan, Pekerjaan, Warganegara, Status Kawin, Golongan Darah, Cacat, Sakit Menahun, Cara KB, dan Status Rekam menjadi teks statis menggunakan array lookup diresources/views/data/penduduk/form_show.blade.php.resources/views/data/penduduk/form_show_status_kehamilan.blade.phpmenjadi tampilan teks statis (Tidak Hamil/Hamil).form-control-static) pada file penduduk gabunganresources/views/data/penduduk/gabungan/form_show.blade.php.Reason for change:
Impact of change:
✅ User Experience: Tampilan detail penduduk menjadi lebih bersih, modern, dan profesional tanpa adanya elemen input abu-abu (disabled inputs).
✅ Performa Rendering: Sedikit peningkatan performa rendering di sisi klien karena browser tidak perlu me-render elemen form interaktif (select, radio button) yang kompleks.
✅ Konsistensi Data: Menghilangkan kebingungan pengguna apakah form tersebut dapat diedit atau tidak.
Related Issue
#1647
Steps to Reproduce
Before fix (problem):
<input>), dropdown (<select>), dan radio button yang semuanya berwarna abu-abu (dinonaktifkan /disabled).After fix (solution):
form-control-static.Testing on related features:
Checklist
Technical Details
Technical Explanation
Perubahan ini berfokus pada file-file presentasi (Blade templates) yang digunakan untuk menampilkan rincian data penduduk.
Inisialisasi Lookup Array: Di bagian atas
form_show.blade.php, kami menambahkan blok PHP untuk memuat data relasi dari model-model database terkait secara efisien menggunakan metodepluck('nama', 'id'):Hal ini memungkinkan kami mengonversi nilai ID/kode yang tersimpan di
$pendudukmenjadi label tekstual yang dapat dibaca manusia tanpa perlu me-render elemen<select>utuh.Penggantian Form Control:
<input type="text" class="form-control" disabled>diganti dengan<p class="form-control-static">untuk memastikan tampilan teks sejajar dengan label form Bootstrap.$radio_sex[$penduduk->sex]dan$radio_status[$penduduk->status].Configuration changes
Tidak ada perubahan konfigurasi.
Dependencies added
No new dependencies
Testing
Manual Testing
Automated Testing
Tidak ada (perubahan hanya pada tingkat visual/Blade template).
Screenshots / Video
Before:
Halaman detail penduduk menampilkan kotak input textbox abu-abu, select dropdown, dan radio button yang dinonaktifkan (disabled).
After:
Halaman detail menampilkan informasi dalam bentuk teks statis yang rapi dan bersih menggunakan class
.form-control-static.Breaking Changes
None
Migration Guide
Not required
References
Additional notes: Perubahan ini murni visual pada file template view blade dan tidak merubah struktur database maupun logika pengontrol (controller).