Closed (fixed)
Project:
Meta tags quick
Version:
7.x-2.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
12 Aug 2011 at 09:28 UTC
Updated:
23 Sep 2011 at 19:11 UTC
Jump to comment: Most recent file
Comments
Comment #1
Sebastian.Buesing commentedCreated a smaller patch. Git apperently just replaced everything in the last patch :-(
Comment #2
Sebastian.Buesing commentedFound a typo and some coding convention violations. Patch should be clean now.
Comment #3
valthebaldLooks very nice, indeed.
New screen already provides added value for module users, however, to make it even more useful, I would add filter option (like in URL aliases list)
Comment #4
valthebaldObviously, there is demand for simplifying mass tags administration.
Please look at another suggestion on at #1241334: Auto settings for Metatags Quick
Comment #5
valthebaldProbably, it's good to spin off nice-to-have, but optional components to the helper module, which can be switched off on production, and used only in dev/staging (similar to views/views_ui separation). What do you think?
Comment #6
davemybes commentedI'll be taking a more in depth look at the GUI stuff tomorrow, but I think the path-based GUI should be part of the core module, as I think 90% of people will use it. The default Metatag settings option could stand on its own, as less people might use it.
Comment #7
davemybes commentedThe GUI works great for me. No issues that I can see.
Comment #8
valthebaldFew code formatting changes, removed (deprecated now) functions that handled until now path-based tags, plus I've changed path based URL from admin/settings/search/metatags_quick/path_based_metatags to admin/settings/search/metatags_quick/path_based - it is already long, and it is clear that path belongs to metatags_quick.
Really nice work done, Sebastien!
Comment #9
valthebald