Problem/Motivation

  • "PurgeUI" is in the filename/classname, yet everywhere else, this is avoided, since it's implied by the namespace.
  • "MainConfigForm" implies there are others too, but there aren't.

Proposed resolution

I'd go for:

  1. ConfigForm if you want to be consistent with the rest of Purge.
  2. PurgeConfigForm if you want to be consistent with core.

Remaining tasks

TBD

User interface changes

None.

API changes

None.

Comments

nielsvm’s picture

I agree, changing

nielsvm’s picture

Status: Active » Fixed

  • nielsvm committed 15f75a7 on 8.x-3.x
    Issue #2425109: "PurgeUiMainConfigForm" is a strange name
    
  • nielsvm committed b838954 on 8.x-3.x
    Issue #2425109: "PurgeUiMainConfigForm" is a strange name
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.