Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Pages: fix placeholder gray bar in date header #11

Closed
alisterscott opened this issue Nov 17, 2015 · 1 comment
Closed

Pages: fix placeholder gray bar in date header #11

alisterscott opened this issue Nov 17, 2015 · 1 comment
Assignees
@gwwar
Labels
Pages [Pri] Low Address when resources are available. [Type] Bug When a feature is broken and / or not performing as intended
Milestone
Core: Iteration 17

Comments

@alisterscott
Copy link
Contributor

When you open the pages page, the clock icon/gray placeholder bar appears whilst the list of pages is loading, which is inconsistent with how other areas in calypso display.

Suggested fix is to not show this icon until loaded.

clock icon shown when loading
@alisterscott alisterscott added [Type] Bug When a feature is broken and / or not performing as intended Pages labels Nov 17, 2015
@mtias mtias changed the title Pages: Placeholder Showing Noticon whilst loading pages Pages: fix placeholder gray bar in date header Nov 17, 2015
@mtias
Copy link
Member

mtias commented Nov 17, 2015

Noting the main problem is the gray placeholder bar within the date header, removing that would already be an improvement even if we keep the icon.

@rralian rralian added this to the Core: Iteration 17 milestone Nov 30, 2015
@hoverduck hoverduck added the [Pri] Low Address when resources are available. label Dec 2, 2015
@gwwar gwwar self-assigned this Dec 12, 2015
@gwwar gwwar mentioned this issue Dec 12, 2015
Merged
ockham pushed a commit that referenced this issue Sep 28, 2018
@akirk @ockham
As per the spec, add verbatim line breaks to the header (#11)
5f9f2f9
ockham pushed a commit that referenced this issue Oct 11, 2018
@akirk @ockham
As per the spec, add verbatim line breaks to the header (#11)
ce99364
sirreal pushed a commit that referenced this issue Dec 5, 2018
@aduth @sirreal
Merge pull request #11 from Automattic/remove/quote-props-rule
455f79e 
remove quote-props rule
sirreal pushed a commit that referenced this issue Dec 5, 2018
@aduth @sirreal
Merge pull request #11 from Automattic/fix/jsx-classname-namespace-no…
581685f 
…n-index

jsx-classname-namespace: Don't consider non-index as root
sirreal pushed a commit that referenced this issue Dec 5, 2018
@aduth @sirreal
Merge pull request #11 from Automattic/remove/quote-props-rule
ceede12 
remove quote-props rule
sirreal pushed a commit that referenced this issue Dec 5, 2018
@aduth @sirreal
Merge pull request #11 from Automattic/fix/jsx-classname-namespace-no…
531216c 
…n-index

jsx-classname-namespace: Don't consider non-index as root
sirreal pushed a commit that referenced this issue Dec 6, 2018
@akirk @sirreal
As per the spec, add verbatim line breaks to the header (#11)
1a913c6
sirreal pushed a commit that referenced this issue Dec 6, 2018
@aduth @sirreal
Merge pull request #11 from Automattic/remove/quote-props-rule
eb226cc 
remove quote-props rule
sirreal pushed a commit that referenced this issue Dec 6, 2018
@aduth @sirreal
Merge pull request #11 from Automattic/fix/jsx-classname-namespace-no…
bcdc6bb 
…n-index

jsx-classname-namespace: Don't consider non-index as root
sirreal pushed a commit that referenced this issue Dec 7, 2018
@akirk @sirreal
As per the spec, add verbatim line breaks to the header (#11)
b967366
sirreal pushed a commit that referenced this issue Dec 7, 2018
@akirk @sirreal
As per the spec, add verbatim line breaks to the header (#11)
4fff461
sirreal pushed a commit that referenced this issue Dec 11, 2018
@akirk @sirreal
As per the spec, add verbatim line breaks to the header (#11)
12fda9b
sirreal pushed a commit that referenced this issue Dec 11, 2018
@aduth @sirreal
Merge pull request #11 from Automattic/remove/quote-props-rule
b527f33 
remove quote-props rule
sirreal pushed a commit that referenced this issue Dec 11, 2018
@aduth @sirreal
Merge pull request #11 from Automattic/fix/jsx-classname-namespace-no…
225e4d7 
…n-index

jsx-classname-namespace: Don't consider non-index as root
sirreal pushed a commit that referenced this issue Dec 11, 2018
@akirk @sirreal
As per the spec, add verbatim line breaks to the header (#11)
ead4536
@stephanethomas stephanethomas mentioned this issue Nov 5, 2020
Merged
@stephanethomas stephanethomas mentioned this issue Dec 7, 2021
Merged
@stephanethomas stephanethomas mentioned this issue Feb 8, 2024
Merged
@mdno mdno mentioned this issue Jan 9, 2023
Open
@snyk-bot snyk-bot mentioned this issue Jul 18, 2020
Open
@anarh anarh mentioned this issue Jan 10, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gwwar gwwar

Labels
Pages [Pri] Low Address when resources are available. [Type] Bug When a feature is broken and / or not performing as intended
Projects
None yet
Milestone
Core: Iteration 17
Development

No branches or pull requests
5 participants
@alisterscott @rralian @mtias @gwwar @hoverduck