Problem/Motivation

Both names are quite long and the Paragraphs prefix is unnecessary.

Proposed resolution

Rename them to StyleDiscoveryInterface and StyleDiscovery

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 rename-2848117-2.patch15.17 KBtbonomelli

Comments

Primsi created an issue. See original summary.

tbonomelli’s picture

tbonomelli commented
Assigned: Unassigned » tbonomelli
Status: Active » Needs review
StatusFileSize
new rename-2848117-2.patch15.17 KB

Removed the Paragraphs prefix from ParagraphsStyleDiscoveryInterface and ParagraphsStyleDiscovery.

tduong’s picture

tduong commented

I was waiting for testbot to pass, but today d.o is pretty slow. Anyway, it looks fine for me :)

Status: Needs review » Needs work

The last submitted patch, 2: rename-2848117-2.patch, failed testing.

tduong’s picture

tduong commented
Status: Needs work » Needs review

Re-run tests.

tduong’s picture

tduong commented
Status: Needs review » Reviewed & tested by the community

I think it's alright.

primsi’s picture

primsi commented
Status: Reviewed & tested by the community » Fixed

Fixed, thanks.

Status: Fixed » Closed (fixed)

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