Skip to content

Add code recipes #86

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
Open
frenzzy wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add code recipes #86

frenzzy wants to merge 1 commit into from

Conversation

@frenzzy
Copy link
Member

@frenzzy frenzzy commented Mar 26, 2017 •
edited
Loading

@frenzzy frenzzy force-pushed the recipes branch 2 times, most recently from 22e4f51 to 6f5d76e Compare April 21, 2017 20:48
@coveralls
Copy link

coveralls commented Apr 21, 2017 •
edited
Loading

Coverage Status

Changes Unknown when pulling 6f5d76e on frenzzy:recipes into ** on kriasoft:master**.

@mksarge mksarge mentioned this pull request Jun 6, 2017
Open
@ImanMh
Copy link

ImanMh commented Aug 16, 2017

how can I use "Controlling scroll behavior" that you mentioned above?
My problem is that I want to change browser location path without refreshing (changing the scroll position) but I don't know if its possible.
(I have a list that I has a sidebar for filtering the list.)

kpaxqin
kpaxqin reviewed Sep 20, 2017
View reviewed changes
docs/isomorphic-routing-with-react-and-express.md Outdated
import routes from './routes';

function render(location) {
UniversalRouter.resolve(routes, location.pathname).then(route => {
Copy link

@kpaxqin kpaxqin Sep 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have this api for now ?
I like this design, for now resolve is an prototype method of class UniversalRouter, and we need to call new UniversalRouter to use resolve

@frenzzy frenzzy mentioned this pull request Oct 18, 2017
Open
@codecov
Copy link

codecov bot commented Jan 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (db8a409) to head (84cb705).
⚠️ Report is 13 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff            @@
##              main       #86   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          229       233    +4     
  Branches        66        73    +7     
=========================================
+ Hits           229       233    +4

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@kpaxqin kpaxqin kpaxqin left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

4 participants

@frenzzy @coveralls @ImanMh @kpaxqin