Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Live preview sometimes ignores CSS changes, especially when Chrome was closed before #2858

Closed
RaymondLim opened this issue Feb 13, 2013 · 11 comments
Assignees
Milestone

Comments

@RaymondLim
Copy link
Contributor

  1. Shutdown Chrome
  2. Open “citrus completed” project or "getting started" project.
  3. Open index.html
  4. Start Live Preview
  5. Put cursor in <body> tag, Ctrl-E
  6. Change background-color

Result: you don't see the update in Chrome.

@redmunds
Copy link
Contributor

Doesn't happen when using a server, so it seems to be a local file permissions problem.

@peterflynn
Copy link
Member

Removing 'Win only' since we now think this happens on Mac too -- it's just timing-sensitive and thus the ease of repro varies from machine to machine.

@peterflynn
Copy link
Member

This also repros in Sprint 19. We believe it actually repros in Sprint 17-18 and Edge Code preview 2 as well. We've also reproduced it at least once in an older version of Chrome.

It's not clear what has changed to make it much easier to repro suddenly, but it's possible the Chrome update on 1/30 altered timing in a way that makes it easier to hit.

gruehle added a commit that referenced this issue Feb 13, 2013
…l page corresponding to the current document (not the interstitial page).
gruehle added a commit that referenced this issue Feb 13, 2013
…l page corresponding to the current document (not the interstitial page).
@ghost ghost assigned gruehle Feb 14, 2013
@pthiess
Copy link
Contributor

pthiess commented Feb 14, 2013

@gruehle: Reviewed - as discussed today this is an exception for Sprint 20. Consequently assigning it to Sprint 21.

@pthiess
Copy link
Contributor

pthiess commented Feb 14, 2013

Marking as Medium priority as this seems somewhat intermittent and actually no data loss.

@njx
Copy link
Contributor

njx commented Feb 25, 2013

Removing from sprint 21--this was moved to https://trello.com/c/R0ZFrFV4

@ghost ghost assigned iwehrman Mar 8, 2013
@njx
Copy link
Contributor

njx commented Mar 8, 2013

This is in sprint 22 as part of the RESEARCH: Live Development Robustness card. No need to add to the bug card. Reassigning to @iwehrman.

@iwehrman
Copy link
Contributor

Some notes on this issue.

@iwehrman
Copy link
Contributor

This was automagically closed by Github because it was mentioned in pull #3142. I'm re-opening it so someone else can confirm properly, but I can't reproduce the issue and do believe that pull fixes it.

@iwehrman iwehrman reopened this Mar 19, 2013
@ghost ghost assigned RaymondLim Mar 19, 2013
@redmunds
Copy link
Contributor

Tagged FBNC and assigning back to @RaymondLim

@RaymondLim
Copy link
Contributor Author

Fix verified.

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

No branches or pull requests

7 participants