Opened 9 years ago
Last modified 9 years ago
#10656 moreinfo_accepted Patch
bookmark category
| Reported by: | ali77 | Owned by: | Stephen L'Allier |
|---|---|---|---|
| Priority: | normal | Component: | FileZilla Client |
| Keywords: | Cc: | ||
| Component version: | Operating system type: | Windows | |
| Operating system version: |
Description
i have many (for example 100) bookmarks in FileZilla for my site.
it is undesirable for me to work with a confused bookmark list :|
i suggest you to add categorize feature to bookmark part of Filezilla.
Attachments (2)
Change History (7)
comment:1 by , 9 years ago
| Owner: | set to |
|---|---|
| Status: | new → accepted |
by , 9 years ago
| Attachment: | bookmarks_dialog_diff.txt added |
|---|
by , 9 years ago
| Attachment: | bookmarks_dialog_h_diff.txt added |
|---|
comment:3 by , 9 years ago
This feature request was worked on as a group project at Florida Institute of Technology for an Intro to Software Engineering class. The group members were: Ash Clarke, Stephen L'Allier, Yannis Lewis, and Trung Nguyen. Yannis primarily worked on this feature request.
comment:4 by , 9 years ago
There are a bunch of issues with the patch:
- Memory leaks
- Missing nullpointer checks
- Hardcoded menu ids
- A using directive
- not passing parameters as constref where possible
- the code operates on site-specific bookmarks. Where are the corresponding changes to the Site Manager?
Please fix these.
comment:5 by , 9 years ago
| Status: | accepted → moreinfo_accepted |
|---|
Note:
See TracTickets
for help on using tickets.
Changing status from bug report to patch. Unified context diff will be attached.