Skip to content

fix(sidebar): drag requests into folders + create new requests inside folders#29

Merged
mertbekci merged 1 commit into
mainfrom
fix/folder-drop
Apr 14, 2026
Merged

fix(sidebar): drag requests into folders + create new requests inside folders#29
mertbekci merged 1 commit into
mainfrom
fix/folder-drop

Conversation

@mertbekci

Copy link
Copy Markdown
Member

Özet

Sidebar workspace ağacında iki kullanılabilirlik hatası düzeltildi:

  • Request'i klasöre sürükle-bırak: Request satırları artık draggable, klasör satırları da drop-target. Bırakıldığında request (ve varsa files/<slug>.body.<ext> sidecar dosyası) yeni klasörün içine taşınır. Drop sırasında klasör otomatik açılır.
  • Klasör içine yeni request: Toolbar + butonu artık "seçili" klasörü parent olarak kullanır (klasöre tıklayınca/sağ-tık yapınca seçili olur). Boş alana tıklamak seçimi sıfırlar. Klasör sağ-tık menüsündeki "New request / New folder" seçenekleri hâlâ çalışıyor.

Değişenler

  • packages/http-core/src/workspace/fs.tsmove() artık request YAML'ındaki file: referanslarını tarıyor, aynı klasördeki sidecar dosyalarını birlikte taşıyor. Hedefte aynı isimli dosya varsa hata atıyor.
  • apps/desktop/src/renderer/src/store.ts — yeni moveNode action; açık bir tab taşınan dosyayı işaret ediyorsa id/relPath güncelleniyor.
  • apps/desktop/src/renderer/src/components/Sidebar.tsx — HTML5 DnD (TabBar ile aynı desen), selectedFolder state, drop highlight, auto-expand.
  • packages/http-core/tests/workspace.test.ts — sidecar move ve duplicate-target testleri.

Test plan

  • pnpm -r typecheck
  • pnpm -r test — 147 passed (önceki 145 + 2 yeni)
  • pnpm --filter=@scrapeman/desktop build
  • Manuel: request'i klasöre sürükle, klasöre tıklayıp toolbar + ile request oluştur

@mertbekci mertbekci added the ready for review PR ready for review label Apr 14, 2026
@mertbekci mertbekci self-assigned this Apr 14, 2026
@mertbekci
mertbekci merged commit bf662dd into main Apr 14, 2026
3 checks passed
@mertbekci
mertbekci deleted the fix/folder-drop branch April 14, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review PR ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant