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.

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

Feat/single page #30

Merged
merged 5 commits into from
Nov 3, 2021
Merged

Conversation

anwari666
Copy link
Contributor

@anwari666 anwari666 commented Nov 2, 2021

Banyak improvement polishing akhir2, termasuk:

  1. implemen single page view
  2. sharable link di twitter
  3. shorter header
  4. simple about di footer
  5. merge conflict with GA stuffs

Will close #26 and hopefully #10 (simpler approach, less dependency)

Screenshot 2021-11-02 at 23 48 34

Copy link
Contributor

@ans-4175 ans-4175 left a comment

Choose a reason for hiding this comment

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

LGTM

"html-webpack-plugin": "^4.5.2",
"mini-css-extract-plugin": "^0.9.0",
"razzle": "^4.0.6",
"razzle-dev-utils": "^4.0.6",
"razzle": "^4.2.6",
Copy link
Contributor

Choose a reason for hiding this comment

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

wiih aman nih naik versi?

@@ -17,6 +17,7 @@ const App = () => {
<Switch>
<Route exact path="/" component={Home} />
<Route exact path="/about" component={About} />
Copy link
Contributor

Choose a reason for hiding this comment

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

about hapusin aja ga sih?

@@ -33,10 +33,12 @@
}

h1 {
font-family: 'TinderFont', cursive;
font-family: 'TinderFont', "Noto Sans", 'Trebuchet MS', sans-serif;
Copy link
Contributor

Choose a reason for hiding this comment

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

oh nice

border: 1px solid var(--light-grey-tw);
}

.button.tweet:hover {
Copy link
Contributor

Choose a reason for hiding this comment

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

nice

@@ -17,6 +17,7 @@ const App = () => {
<Switch>
<Route exact path="/" component={Home} />
<Route exact path="/about" component={About} />
<Route exact path="/:profileId" component={Home} />
Copy link
Contributor

Choose a reason for hiding this comment

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

weh baru sadar disatuin yap

@@ -13,7 +13,7 @@ function pickRandom(arr, n) {
result[n] = arr[x in taken ? taken[x] : x];
taken[x] = --len in taken ? taken[len] : len;
}
return result.map( res => ({...res, id: short.generate() }));
return result.map( res => ({...res, unique_key: short.generate() }));
Copy link
Contributor

Choose a reason for hiding this comment

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

sepakat

<h2 className='infoText'>
{lastDirection ? `` : 'Geser gambar untuk memulai'}
</h2>
{ !lastDirection && <h2 className='infoText'>Geser gambar untuk memulai</h2>}
Copy link
Contributor

Choose a reason for hiding this comment

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

oh iya bener begini


const allGirls = data && ('pages' in data) ? data.pages.flat() : [];
const allGirls = data && ('pages' in data)
Copy link
Contributor

Choose a reason for hiding this comment

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

urang ga ngerti baca ini, jadi kalau ada profile, si profile ditaruh paling atas kan? tapi di belakangnya tetap dari infiniteQuery kan?

const allGirls = data && ('pages' in data) ? data.pages.flat() : [];
const allGirls = data && ('pages' in data)
? profile
? [...data.pages.flat(), profile]
Copy link
Contributor

Choose a reason for hiding this comment

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

cool sih

const swiped = (direction, id) => {
const girl = allGirls.find((item) => item.id === id);
const swiped = (direction, key) => {
const girl = allGirls.find((item) => item.unique_key === key);
Copy link
Contributor

Choose a reason for hiding this comment

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

okay sepakat

@ans-4175 ans-4175 merged commit d624dd0 into begundal-ug:master Nov 3, 2021
@ans-4175 ans-4175 mentioned this pull request Nov 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refine tombol dalam page dan Membuat tampilan single page
2 participants