https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15707
--- Comment #63 from Kyle M Hall <[email protected]> --- > I agree. It is confusing to have two menu items on the administration page, > "Libraries and groups" and "Library groups." I think we should think of a > way to manage all this functionality in one section. These two sub-systems are completely unrelated. They really don't belong together. I'd be more than happy to rename it "Library hierarchies" if you find that acceptable, or to rename the existing groups to "search groups" which is what they are. > Some other comments: > > - Nowhere else do we use the "fa-plus-circle" icon for "add." We always use > "fa-plus." Changed with followup. > > - Nowhere else in Koha do we use a modal to display an add form. We either > display a form on a separate page or we use JS to show/hide the edit form on > the page to mimic the display of a separate page. See > admin/items_search_fields.pl for example. You are correct, but I believe the modal may be a better paradigm. Would you disagree? It means we aren't shifting elements on the page around when we hide/display a form. I know it's without precedent at the moment but I'd like you to consider the advantages. Think about how every other site does this stuff. It's typically with modals. Koha is generally working against the standard web ux by *not* using modals! Would you agree? > > - Nowhere else in Koha do we use the "fa-edit" icon for "edit." We always > use "fa-pencil." Changed with followup! > > - When displaying confirmation messages we use '<div class="dialog > message">' not '<div class="dialog alert">' Can do! > > - There is no reason to display an icon ("<i class="fa > fa-object-group"></i>") next to each group. It doesn't serve any function. Changed in followup! > > - There is no reason to display an icon ("<i class="fa fa-home"></i>") next > to each library in a group. It doesn't serve any function. Changed in followup! > > - Nowhere else in Koha do we use "fa-minus-circle" to mean "remove." It > should be "fa-trash." Fixed in followup! > > - Nowhere else in Koha do we add an icon to the "actions" menu. Will do! > - Nowhere else in Koha do we use a modal to display a delete confirmation > message. I think I could be convinced that we should, but I'd like to hear > the argument. If we do, we should use the space given to display more > information about the deletion than this implementation does, e.g. what > exactly am I deleting? We use plenty of js confirm dialogs for delete confirmation messages. Over the years they've gotten more and more jarring in my opinion. They cannot be styled to match the look of Koha. Even more importantly, we tend to abuse confirm dialogs by adding far to much text to them which we attempt to lay out with only spaces and newlines. With the modal this is a non-issue because we can simply use html. The modal seems much more natural and integrated, while presenting the exact same options to the user. > > - Instead of a "home" icon for "Add library" I think it should be the "plus" > icon which is associated with adding or creating something. Fixed! > > - The list of libraries in the "Add library" dropdown should be alphabetized > by library name, not library code. Fixed! > - This patch adds five new instances of "onclick." I've been working very > hard to eliminate the use of "onclick" and I don't think new patches should > add them. Fixed! I wrote this patch before the move away from onclick and I'm happy to see them go ; ) -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
