[drupal-devel] [task] Improvments to Filter UI

Bèr Kessels drupal-devel at drupal.org
Fri Jul 29 09:04:43 UTC 2005

Issue status update for 
Post a follow up: 

 Project:      Drupal
 Version:      cvs
 Component:    filter.module
 Category:     tasks
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  Bèr Kessels
 Updated by:   Bèr Kessels
 Status:       patch
 Attachment:   http://drupal.org/files/issues/filter_module_UI_consistancy_HEAD_0.patch (10.37 KB)

Dries, thanks a lot for the review. OI fixed your comments. Or so I
hope. code-style.pl was of no use (can that *please*  be removed from
core?) because it chokes on the help texts. It also pointed out a lot
of old formatting isues, whic, when fixed wuold flood this patch with
unrelated fixes. I tried to narow my style-searching to the areas I
touched. And I made the comments more verbous. 

The query is fixed. it saves no characters, but looks better now.
Thanks for the tip. 

The print_r..... blush. thats what I get for not running my default
grep -r print_r on my code after finishing up...

I could not find a lot of spaces and tabs. So it lmight be that i
missed one. I triple checked it though.

Bèr Kessels

Previous comments:

Sun, 24 Jul 2005 09:30:54 +0000 : Bèr Kessels

This is a mockup for the Filter UI. Currently it is soo complex, that I
dare to call it broken. 

Please comment on this, for I will not ake any patches, when the
general idea is disliked.

The idea is t split filters and input formats better. Filters are
filters, defined in modules. Input formats are bundles of filters. This
is how we have it ATM, but the interface fails to communicate that. I
tried to develop a consistent (with the rest of Drupal) interfce that
makes it clearer what is what. 

The menu is as follows:
* administer
** ....
** settings
*** input formats
*** filters
** ...


Sun, 24 Jul 2005 09:37:10 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/input-formats_list1.png (16.34 KB)

* administer
** ....
** settings
*** input formats >> see attachement 1
*** filters


Sun, 24 Jul 2005 09:39:02 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/input-formats_add.png (23.42 KB)

* administer
** ....
** settings
*** input formats >> see attachement 2, tab 2
*** filters

Note: The default checkbox lmay seem a bit odd. But checking it will
remove the "default status" from the current "default" one and make
this one "default". The help text, and a drupal_set_message should
explain this.


Sun, 24 Jul 2005 09:40:08 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/input-formats_edit_format.png (22.17 KB)

* administer
 ** ....
 ** settings
 *** input formats >> clicking a 'configure' link in the table of
attachement 1
 *** filters


Sun, 24 Jul 2005 09:40:55 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/input-filters_list.png (36.24 KB)

* administer
 ** ....
 ** settings
 *** input formats
 *** filters >> see attachement 4


Sun, 24 Jul 2005 09:41:40 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/input-formats_edit.png (22.63 KB)

* administer
 ** ....
 ** settings
 *** input formats
 *** filters >> clicking a 'configure' link in the table of attachement


Mon, 25 Jul 2005 14:46:27 +0000 : Bèr Kessels

IMO this makes the interface easier. But other disagree. And because it
also removes one feature: (being able to use E.g. HTML in different
input formats, with different settings)
I'll simply wont fix this :(


Thu, 28 Jul 2005 13:36:12 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/filter_module_UI_consistancy.patch (10.28 KB)

I decided to go for it anyway. Be it in a slightly different way then my
initial mockups. The difference is that I left the filters where they
are, hidden beneath the formats.

So, here is the patch that makes the filter UI more consistent with
screens like blocs, menus, vocabularies et al

also nice to note is that:
93 lines are added, 104 are removed. So netto we have less code :)
(cvs diff filter.module | grep ^+ | wc -l)


Thu, 28 Jul 2005 13:37:02 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/input-formats_list_html.png (44.64 KB)

Here is how the listing now looks.


Thu, 28 Jul 2005 13:37:57 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/input-formats_edit_format_html.png (113.71 KB)

And the "configure" screen.
The 'add' screen looks exactly the same, only with the default values


Thu, 28 Jul 2005 13:39:19 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/input-formats_edit_format_too_long.png (121.82 KB)

Oh, And here is a really nice example of why the current screen is no
And here I "only" have four roles. I run a site with 12 roles, imagine
this screen then!


Thu, 28 Jul 2005 13:48:35 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/filter_module_UI_consistancy_HEAD.patch (10.24 KB)

And here is a pathc for HEAD (I forgot that I developed this on a 4.6.2
codebase. Sorry)


Thu, 28 Jul 2005 13:55:25 +0000 : stefan nagtegaal

I like this very, very much!
This is - indeed - much better than the current UI. This always fits,
and makes the life of people who use a fixed width theme much nicer..

Ber, FYI (you probably know this too), 'admin/access' does also break
fixed width themes due to the fact that it expands the width of the
table when more roles are being added..

Good catch! I like it much more this way..

+1 from me...


Fri, 29 Jul 2005 07:27:34 +0000 : Dries

The code style needs work (spaces, tabs, 'as' -> 'AS'), the code
comments need work, and debug statements should be removed. 
Screenshots look good to me.  db_query('UPDATE {filter_formats} SET
cache = %d, name = \'%s\', roles = \'%s\' WHERE format = %d',
(int)$cache, $name, $roles, $format); can be written better/shorter

More information about the drupal-devel mailing list