Matt Westcott

Results 492 comments of Matt Westcott

@thibaudcolas Assigning you for review on this... hopefully the updates to the testing infrastructure are uncontroversial enough, but would really appreciate your input on whether https://github.com/wagtail/wagtail/pull/12203/commits/5dab8c86b36b537d4a1b7f583bccd4153944b567 is a legitimate fix...

> (actually I see the 'delete' buttons are missing an aria-label, which I'll fix shortly) Sorry, scratch that... I was thinking we could give them a label like "Delete permission...

Some assorted not-fully-formed thoughts, about this PR but also the situation that got us here: It kind of sucks that a developer has to know the correct FooButton class to...

Confirmed on current Wagtail main. Steps to reproduce: ``` wagtail start mysqlsearch cd mysqlsearch pip install mysqlclient ``` in settings/base.py, change DATABASES to ``` DATABASES = { "default": { "ENGINE":...

The out-of-the-box search functionality from the project template does not use `parse_query_string` - it [passes the entered search query straight to `.search()` as a string](https://github.com/wagtail/wagtail/blob/4f36562de7f1461de6202f4bd44b58cbbee00833/wagtail/project_template/search/views.py#L9-L14) - so the expected behaviour...

A minimal snippet to reproduce the error, replicating what the search backend is doing internally: ```python from wagtail.search.backends.database.mysql.query import Lexeme, MatchExpression, SearchQuery from wagtail.search.models import IndexEntry from django.db.models.fields import BooleanField...

Makes sense... I'd go along with that. As a slight refinement - for people making use of the [subpage_types](http://wagtail.readthedocs.org/en/latest/core_components/pages/creating_pages.html?highlight=subpage_types#wagtail.wagtailcore.models.Page.subpage_types) feature - we could have it so that it adds a...

Thanks for the feedback @BigWhale! I suspect - albeit without any hard evidence - that the pattern of publishing a page and then working on it some more is the...