Skip to content

[4] Rename recent BlankState feature to EmptyState#33334

Merged
HLeithner merged 6 commits intojoomla:4.0-devfrom
PhilETaylor:renameblankstate
Apr 27, 2021
Merged

[4] Rename recent BlankState feature to EmptyState#33334
HLeithner merged 6 commits intojoomla:4.0-devfrom
PhilETaylor:renameblankstate

Conversation

@PhilETaylor
Copy link
Copy Markdown
Contributor

The concept we are implementing is called Empty State design, therefore it makes sense to use that term throughout and not to confuse people with "Blank state/Blank slate" terms. Sorry, that was my fault.

Therefore at this early stage it makes sense to get our own house in order from the start and use the term "Empty State" to refer to the "Empty State Design UX" we are employing.

References (just a quick google):

@richard67
Copy link
Copy Markdown
Member

That's the rename of the layouts. But what about these $this->get('IsBlankSlate') calls in the if conditions? Will the IsBlankSlate be renamed, too?

@PhilETaylor

This comment was marked as abuse.

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Apr 26, 2021

Please rename the language constants too.

@joomla-cms-bot joomla-cms-bot added the Language Change This is for Translators label Apr 26, 2021
@PhilETaylor

This comment was marked as abuse.

@PhilETaylor

This comment was marked as abuse.

@PhilETaylor

This comment was marked as abuse.

Phil E. Taylor and others added 5 commits April 26, 2021 18:42
Signed-off-by: Phil E. Taylor <phil@phil-taylor.com>
Signed-off-by: Phil E. Taylor <phil@phil-taylor.com>
Signed-off-by: Phil E. Taylor <phil@phil-taylor.com>
a-z
Signed-off-by: Phil E. Taylor <phil@phil-taylor.com>
@PhilETaylor

This comment was marked as abuse.

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Apr 26, 2021

feeling guilty... 😨
but this task cannot be automated ? (unit/system) test

@PhilETaylor

This comment was marked as abuse.

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Apr 26, 2021

nevermind
i've liked the idea,
didn't suspected to open a small pandora box...
me too lazy

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Apr 26, 2021

I have tested this item ✅ successfully on 6ed02ba


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33334.

@Bakual
Copy link
Copy Markdown
Contributor

Bakual commented Apr 26, 2021

it was the great work by @Bakual that messed them up

You're welcome 🤣

@PhilETaylor

This comment was marked as abuse.

@Bakual
Copy link
Copy Markdown
Contributor

Bakual commented Apr 26, 2021

It was a pleasure (really!) :)

@joomdonation
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on 6ed02ba


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33334.

@richard67
Copy link
Copy Markdown
Member

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33334.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 27, 2021
@richard67 richard67 added this to the Joomla 4.0 milestone Apr 27, 2021
@HLeithner HLeithner merged commit 2559e12 into joomla:4.0-dev Apr 27, 2021
@HLeithner
Copy link
Copy Markdown
Member

Thanks

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Apr 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language Change This is for Translators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants