Githawk: Filter menurut label (Daftar Masalah Repo)

Pergi untuk itu!

masuk akal @Huddie. Saya sering menggunakan fitur penyadapan label di web GitHub untuk mencari masalah "dicari bantuan". Apakah Anda mengambil ini? Saya akan senang untuk mengerjakannya jika Anda tidak.

@BrianLitwin Saya agak sibuk dengan sekolah beberapa hari ke depan jadi saya tidak akan bisa melakukannya. Jika Anda ingin menjatuhkannya, itu akan luar biasa!

_{Dikirim dengan GitHawk}

Dari Masalah:

Dari tumpukan pengontrol tampilan yang lebih dalam:

@BrianLitwin

1. Tampak menakjubkan
2. Ini juga berfungsi pada repoIssueSectionController?

_{Dikirim dengan GitHawk}

Ini juga berfungsi pada repoIssueSectionController?

@Huddie Saya akan melakukan PR ini tanpa fitur itu -> Label ini bersarang cukup dalam dan saya ingin memastikan pola komunikasi yang saya gunakan memenuhi harapan terlebih dahulu. Kemudian menambahkan bahwa dalam akan lebih mudah.

@BrianLitwin @rnystrom Saya pikir mengklik label di repoIssueSectionController adalah yang penting hanya karena di situlah pengguna mencari masalah dan ingin memfilter pencarian itu. Kecil kemungkinan seseorang akan bersarang jauh di suatu tempat yang ingin mencari dengan label.
Saya tahu pada awalnya saya menyarankan agar kita mengarahkan ulang ke pencarian ketika mengklik label di luar repoIssueSectionController tapi saya pikir kita harus tetap menggunakan Github latihan yang satu ini:

Di dalam repoIssueSectionController -> Menambahkan label:xxxxx ke bilah pencarian
Di tempat lain -> Menampilkan deskripsi label atau jika tidak ada menunjukkan nama label.

Saya akan menyentuh yang "Di mana saja". Jika Anda menggunakan github dan mengarahkan kursor ke label, ia memiliki tampilan kursor yang menyebutkan nama label. Hanya ketika di dalam tab masalah, saat mencari masalah, itu ditambahkan ke pencarian.
Saya tidak berpikir mengarahkan orang ketika mereka mengklik label (terutama ketika dalam) adalah ide yang bagus. Bisa mengganggu jika Anda melakukannya secara tidak sengaja

Pikiran?

Saya tidak berpikir mengarahkan orang ketika mereka mengklik label (terutama ketika dalam) adalah ide yang bagus. Bisa mengganggu jika Anda melakukannya secara tidak sengaja

kami sudah mengarahkan ulang mereka ke pencarian tampilan web, jadi coba saja ganti dengan pencarian dalam aplikasi. Saya setuju bahwa ada banyak area permukaan yang 'dapat diklik' pada UI dan penting untuk tidak mengganggu pengguna.

Juga setuju bahwa pengguna mungkin ingin/mengharapkan fungsi labelPressed -> pencarian Masalah terjadi di pengontrol tampilan masalah awal, saya tidak keberatan PR'ing fungsi itu sebagai gantinya.

Bagaimana dengan mendorong pengontrol pencarian baru saat mengetuk label alih-alih muncul kembali ke pengontrol terdekat? Dikombinasikan dengan kemampuan untuk mengetuk label di hasil pencarian, ini akan lebih cocok dengan perilaku GitHub.

️ saran saya tetap hanya vc pencarian yang memiliki label yang dapat diklik dan yang lainnya hanya menampilkan deskripsi atau nama label. Saya pikir mendorong mungkin lebih masuk akal daripada muncul tetapi mendorong dapat menyebabkan setumpuk penuh vc pencarian dan masalah vc kecuali saya salah memahaminya.

IMO tumpukan itu diharapkan karena begitulah perilaku versi browser. Anda dapat menekan "kembali" beberapa kali untuk kembali ke notifikasi, atau UI dapat diubah sehingga bilah tab selalu terlihat dan pengguna dapat mengklik ikon yang sesuai untuk kembali ke notifikasi.

Peramban memiliki ciri-ciri tertentu yang membuatnya masuk akal. Yang utama adalah bagaimana Anda keluar dari tumpukan dan kembali ke aplikasi utama. Seperti yang Anda sarankan, kami dapat memindahkan tabbar ke overlay. Sepertinya perubahan yang lebih besar yang harus ditimbang oleh @rnystrom

_{Dikirim dengan GitHawk}

IMO ini seharusnya hanya mendorong VC pencarian lain dengan filter baru dan tidak muncul kembali. Itu sedikit pengalaman yang menggelegar IMO.

Jika kami khawatir memiliki bilah pencarian yang dapat diedit, mungkin kami menambahkan opsi untuk membuat pencarian VC dapat dicari atau "diperbaiki".

Tapi saya baru saja mulai dengan mendorong VC pencarian baru.

@rnystrom untuk membiarkan orang mengetuk label di repoIssueSectionList (daftar RepositoryIssuesVC utama), kami masih ingin mendorong pengontrol tampilan baru? Itu masuk akal bagi saya, hanya memeriksa ulang.

@BrianLitwin Mengapa kita mendorong? Saya pikir itu hanya harus mengedit pencarian saat ini.

_{Dikirim dengan GitHawk}

@Huddie Anda benar, kami juga bisa melakukannya. Secara khusus, saya berpikir bahwa akan lebih mudah untuk kembali ke status default RepositoryIssuesVC dengan menekan kembali dari Pushed VC, daripada dengan mengedit bilah pencarian. Anda melihat apa yang saya katakan? Itu mungkin bukan asumsi yang bagus.

ya sulit untuk mendapatkan hasil yang diharapkan dari memperbarui bilah pencarian, tetapi juga memberikan cara cepat untuk kembali. Saya masih merasa bahwa perilaku yang diharapkan untuk mengisi pencarian daripada mendorong VC baru lebih masuk akal. Kalau tidak, kami mendorong vc baru yang persis sama dengan satu perbedaan, teks. ️. Mungkin tombol hapus label atau sesuatu untuk membuat penghapusan label dari bilah pencarian lebih cepat?

_{Dikirim dengan GitHawk}

@BrianLitwin ada pembaruan pada label di repoIssueVC? Haruskah masalah ini ditutup?

_{Dikirim dengan GitHawk}

@Huddie agak menunggu @rnystrom untuk mempertimbangkan mendorong VC baru versus mengedit bilah pencarian dalam kasus terakhir mengetuk label masalah di RepoIssuesV utama

Ups. @BrianLitwin mendorong mungkin yang paling sederhana untuk bergerak? Tetapi jika mengedit di tempat itu mudah, itu akan keren!

Saya ingin melacak detail tambahan dari fitur ini, yang mencoba memberi pengguna umpan balik touchDown saat mereka mengetuk label sehingga mereka memahami apa yang terjadi saat transisi tampilan.. mungkin menggelapkan font atau meredupkan latar belakang IssueLabel?

Bagaimana dengan mengurangi alpha-nya?