Skip to content

Change Pokes interface#891

Merged
homu merged 1 commit intonew-xkit:masterfrom
Wolvan:features/pokes/implement_pokes_menu_and_list
Dec 30, 2015
Merged

Change Pokes interface#891
homu merged 1 commit intonew-xkit:masterfrom
Wolvan:features/pokes/implement_pokes_menu_and_list

Conversation

@Wolvan
Copy link
Member

@Wolvan Wolvan commented Dec 29, 2015

Since there are many things planned for Pokes, the interface in the
settings panel was not going to be sufficient for it all. The Pokemon
list has now been moved into it's own window accessible from a menu
in the Pokes settings.

This brings us a step closer to completing #855
Closes #854

Copy link
Member

Choose a reason for hiding this comment

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

You forgot a semicolon here.

@Wolvan Wolvan force-pushed the features/pokes/implement_pokes_menu_and_list branch from f2bff17 to b880f83 Compare December 29, 2015 12:26
@Wolvan
Copy link
Member Author

Wolvan commented Dec 29, 2015

@ThePsionic that's what I get for coding at 3AM.
fixed now

@invalidCards
Copy link
Member

a
In my opinion the gender symbol in the description is a bit small, and the gender text is in the default system font rather than one specified by us.

@invalidCards
Copy link
Member

Also, gender icons next to Pokémon on the dashboard are kinda small now

@Wolvan
Copy link
Member Author

Wolvan commented Dec 29, 2015

@ThePsionic I don't experience the issues you have, for some reason

@invalidCards
Copy link
Member

Note that I'm using Chrome on Linux Mint.

@invalidCards
Copy link
Member

Never mind, happens in 0.4.1 too.

@Wolvan
Copy link
Member Author

Wolvan commented Dec 29, 2015

@ThePsionic good to go then?

Copy link
Member

Choose a reason for hiding this comment

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

Not sure what's supposed to show up here, but it doesn't show up.

Copy link
Member Author

Choose a reason for hiding this comment

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

The thing is display: none that keeps it from showing up. The icon is supposed to be the edit pen that tumblr uses on posts you can edit if you know what I mean. I did not implement nicknaming yet so I decided to hide the symbol until Nicknames become a thing

Since there are many things planned for Pokes, the interface in the
settings panel was not going to be sufficient for it all. The Pokemon
list has now been moved into it's own window accessible from a menu
in the Pokes settings.
@Wolvan Wolvan force-pushed the features/pokes/implement_pokes_menu_and_list branch from b880f83 to 3bfe384 Compare December 30, 2015 12:43
@Wolvan
Copy link
Member Author

Wolvan commented Dec 30, 2015

@ThePsionic I did the fixing magic thingie

@invalidCards
Copy link
Member

LGTM, although I do really think we should use a webfont for the symbols to make sure the same experience is delivered across platforms.

@Wolvan
Copy link
Member Author

Wolvan commented Dec 30, 2015

@ThePsionic should we merge this and then decide on a webfont?

@nightpool
Copy link
Member

They should be very standard symbols—iirc they're even code page 437
symbols, so support shouldn't really be a problem.
On Wed, Dec 30, 2015 at 11:03 AM Wolvan notifications@github.com wrote:

@ThePsionic https://github.com/ThePsionic should we merge this and then
decide on a webfont?


Reply to this email directly or view it on GitHub
#891 (comment).

@invalidCards
Copy link
Member

@Wolvan Fair enough.
@homu r+

@homu
Copy link

homu commented Dec 30, 2015

📌 Commit 3bfe384 has been approved by ThePsionic

@homu
Copy link

homu commented Dec 30, 2015

⚡ Test exempted - status

@homu homu merged commit 3bfe384 into new-xkit:master Dec 30, 2015
homu added a commit that referenced this pull request Dec 30, 2015
…ist, r=ThePsionic

Change Pokes interface

Since there are many things planned for Pokes, the interface in the
settings panel was not going to be sufficient for it all. The Pokemon
list has now been moved into it's own window accessible from a menu
in the Pokes settings.

This brings us a step closer to completing #855
Closes #854
@Wolvan Wolvan deleted the features/pokes/implement_pokes_menu_and_list branch December 30, 2015 17:28
Copy link
Member

Choose a reason for hiding this comment

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

should be specified in a hex format like the rest of the symbols

Copy link
Member Author

Choose a reason for hiding this comment

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

I did not know what the hex format was. I copied it from tumblr's CSS file

Copy link
Member

Choose a reason for hiding this comment

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

a

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.

Rethink Pokés interface

6 participants