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

Conversation list mobile #5813

Merged

Conversation

chrisnojima
Copy link
Contributor

@keybase/react-hackers this gets the conversation list dumb rendering done

screen shot 2017-02-10 at 3 14 45 pm

@keybase-ci-visdiff
Copy link

The commits 6f75f65...dd06168 introduce visual changes on linux.

🔎 9 changed

@keybase-ci-visdiff
Copy link

The commits 6f75f65...a9eb9bc introduce visual changes on linux.

🔎 9 changed

return exists
? <Emoji {...this.props} />
: <Text
type='Body'
Copy link
Contributor

Choose a reason for hiding this comment

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

Indent

@@ -8,10 +8,10 @@ import {envVarDebugJson, isTesting} from './local-debug'

// Shared settings
const dumbFilterJson = (envVarDebugJson() || {}).dumbFilter || ''
const dumbFilterOverride = '' // Changing this will apply during a hot reload session
const dumbFilterOverride = 'conversations' // Changing this will apply during a hot reload session
Copy link
Contributor

Choose a reason for hiding this comment

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

✂️


export const dumbFilter = dumbFilterOverride || dumbFilterJson

// Mobile only settings
export const dumbIndex = 0
export const dumbIndex = 9
Copy link
Contributor

Choose a reason for hiding this comment

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

✂️

@chrisnojima chrisnojima merged commit 0694628 into nojima/DESKTOP-rn-fixes-4 Feb 14, 2017
@chrisnojima chrisnojima deleted the nojima/DESKTOP-conversation-list-mobile branch February 14, 2017 14:37
chrisnojima added a commit that referenced this pull request Feb 14, 2017
#5812)

* added watchman config. added dummy chat component files. fix text eating taps. rename yourekey compnent. add simulator flag to native. add notification listeners

* dumb chat cross platform

* conversation list refactor

* undo whitespace

* lint fixes

* plumb through backgroundColor

* Conversation list mobile (#5813)

* usernames working

* bottom line working

* WIP

* fixup merge

* fix overflow on row bottom

* pr fixes

* min width on convo list

* WIP

* lineClamp messes up truncation for some reason
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.

None yet

3 participants