Skip to content
This repository has been archived by the owner on Feb 5, 2022. It is now read-only.

bug(global - navigation): page flickers on-load when navigating between pages #16

Open
Wholloran opened this issue May 3, 2018 · 6 comments
Labels
open Good to start work on

Comments

@Wholloran
Copy link
Contributor

The page will flicker when navigating between tabs(Home, About, Services, and Contact). This seems to be present on all browsers.

@jedihacks
Copy link
Member

Fixed this in branch https://github.com/openforge/main-website/commits/website-server-enhancements

However; with the fix (and using proper Stencil routing) we expose some other problems that @mmalcorn mentioned.

One of them is that with this fix we see that from the Home or About page to the Contact or Opportunities page; the header goes away.

I identified the issue here; it's related to the CSS naming convention and this code

  let element;
    try {
      element = document.querySelector('header.hero');
    } catch (e) {
      console.log('completed app-hero undefined', e);
    }
    console.log(
      'Element = ',
      element,
      ' with background ',
      `url('${this.backgroundUrl}')`
    );
    element.style.backgroundImage = `url('${this.backgroundUrl}')`;

If you inspect it when going to the page directly; you'll see that it's injecting the CSS into the wrong element.

Will fix this tomorrow with @LizCottrell or @PaulPaulDevelops

@PaulPaulDevelops
Copy link
Member

Looking into this! Will commit to website-server-enhancements

LizCottrell pushed a commit that referenced this issue May 21, 2018
broke out component into individual hero HTML + styles for each page (contact, about, opportunities)

#16
@Wholloran
Copy link
Contributor Author

The flicker is still present for the first time you switch between each Tab while serving, however using the Heroku URL The flicker is still constantly there.

@Wholloran Wholloran removed the QA ready label Jun 1, 2018
@mmalcorn mmalcorn removed their assignment Jun 8, 2018
@LizCottrell LizCottrell changed the title Page will flicker when navigating between tabs bug(global): page flickers on-load when navigating between pages Jun 11, 2018
@LizCottrell LizCottrell added backlog Not approved for development and removed bug labels Jun 11, 2018
@LizCottrell LizCottrell changed the title bug(global): page flickers on-load when navigating between pages bug(global - navigation): page flickers on-load when navigating between pages Jun 11, 2018
@nbey nbey added In-Progress Fix is actively being worked on and removed backlog Not approved for development labels Jun 12, 2018
@nbey
Copy link
Contributor

nbey commented Jun 12, 2018

I have not been able to find a solution, and I'm not sure if this issue is directly related to what @jedihacks previously resolved.

@nbey nbey added open Good to start work on and removed In-Progress Fix is actively being worked on labels Jun 12, 2018
@Wholloran
Copy link
Contributor Author

After discussion lets try a few things, let's create a sandbox project to reproduce and play with. Then reach out to the ionic team and see if they might have any ideas. Let's also Test out the stencil version pump branch and see if the issue is still present there( it might already be resolved)

jedihacks added a commit that referenced this issue Jul 15, 2018
Not a permanent fix, but removes footer from flash and keeps header involved.  Also, clean up and
refactor

re #16
@jedihacks
Copy link
Member

Not fixed; but mitigated with this commit: cd33ee0

I believe the issue may have to do with us performing some conditional rendering. See how the header shifts slightly, as if it's being resized? I think that's our clue

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
open Good to start work on
Projects
None yet
Development

No branches or pull requests

6 participants