Clarify recurrence-interval

  • Resolved Daedalon

    (@daedalon)


    13 years, 1 month ago

    Recurrence interval currently reads like this:

    This event repeats [Monthly] every [1 ] month on the [first ] [Wed] of each month

    We have gotten complaints of this being confusing, and we have to agree, both on readability and clarity. It also leads to strikingly impossible statements: “every [2 ] month … of each month”. Here’s my suggestion:

    This event repeats [Monthly] on the [first ] [Wed] of [each] month

    Rationale and benefits:

    1. More human-readable.
    2. No more errors with month vs. months, for which every language has their own rules.
    3. “Every [ ] day/week/month/year” does not need an unlimited amount of choices. In the spirit of not creating unnecessary options in the UI I would start with three options, “each”, “every second” and “every third”, and add more only if feedback warrants it.
    4. PHP, CSS, HTML, translations and JS become easier to maintain. I’ve had to study the current solution thoroughly in order to make customizations and it’s particularly tricky.

    http://wordpress.org/extend/plugins/events-manager/

Viewing 5 replies - 1 through 5 (of 5 total)
  • agelonwl

    (@angelonwl)

    13 years, 1 month ago

    thanks, maybe @marcus does have better idea about this issue.

    Plugin Author Marcus (aka @msykes)

    (@netweblogic)

    13 years, 1 month ago

    i don’t agree with this, because it makes less sense to the majority I think. I’d say this needs to be a custom template on your side, unless others agree with this.

    plus,

    This event repeats [dakly] of [x] days

    makes no sense, the above works for everything.

    Thread Starter Daedalon

    (@daedalon)

    13 years, 1 month ago

    I must have expressed the proposed format unclearly. “of [x] days” is not part of my idea at all. These are:

    This event repeats [Daily] [every] day

    This event repeats [Weekly] on [ ] Mon [ ] Tue [ ] Wed [ ] Thu [ ] Fri [ ] Sat [ ] Sun [every second] week

    This event repeats [Monthly] on the [second] [Wed] of [every third] month

    Thread Starter Daedalon

    (@daedalon)

    13 years, 1 month ago

    One of the many reasons to do it like this is the same as why you have done the monthly recurrence as “[first|second|third|fourth|last]” instead of “[1 ].”: it’s much more clear and readable this way. The proposed change is to use this way with all recurrence intervals instead of just a part of the monthly.

    Plugin Author Marcus (aka @msykes)

    (@netweblogic)

    13 years ago

    I still don’t agree, to me it makes more sense the way it is, plus your way is more complicated code-wise to get right. Looking at other calendars the repeat every x months/days etc. is done this way.

Viewing 5 replies - 1 through 5 (of 5 total)

The topic ‘Clarify recurrence-interval’ is closed to new replies.