Dim blocked tasks in org-ql-view

[akirak]

This PR changes org-ql-search and org-ql-view to respect the value of org-agenda-dim-blocked-tasks variable, as in org-agenda. It works in both situations where the value is t and invisible.

Once these properties are added to text, org-agenda-dim-blocked-tasks function which is called from org-agenda-finalize-entries handles the rest.

[akirak]

This implements #105 (comment).

[akirak]

@alphapapa This is an alternative to #190 which doesn't hide blocked entries but dim them.

[alphapapa]

@alphapapa This is an alternative to #190 which doesn't hide blocked entries but dim them.

I'm not sure what you mean. Having a blocked predicate seems useful, regardless of dimming such tasks in view buffers.

[akirak]

Having a blocked predicate seems useful, regardless of dimming such tasks in view buffers.

Yes, it would be possible to turn on this option to dim blocked entries, while hiding them using the blocked predicate, but it won't take any effect on actual presentation. If you show only blocked entries, this option will dim all items in the buffer, which again makes no sense. Therefore the blocked predicate and this dimming option should be used mutually exclusively, even if it is technically possible to use both of them. This is what I meant by alternative. The user can either hide blocked entries entirely using the predicate or just dim them via this option.

[alphapapa]

Thanks for your patience.

Looking at this again, I'm not sure if I want to implement this feature, or at least not in this way. I'd prefer not to add support for all the org-agenda- special variables that modify its appearance or add features. I'm hoping to make the org-ql-view implementation cleaner and more flexible.

An alternative might be to have a list of functions that could modify a string in this way, and users who want to dim blocked tasks could add the function to the list.

What do you think? Thanks.

[akirak]

I'd prefer not to add support for all the org-agenda- special variables that modify its appearance or add features.

Makes sense.

An alternative might be to have a list of functions that could modify a string in this way, and users who want to dim blocked tasks could add the function to the list.

That would be a good idea, as it would allow extra flexibility.

Actually, I don't need this feature right now, so you can close this PR. Thank you for your consideration.

[alphapapa]

That's ok, I'll leave it open as an idea for future consideration. Thanks.