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

Add server browser from ingame #1636

Merged
merged 1 commit into from Aug 18, 2019
Merged

Add server browser from ingame #1636

merged 1 commit into from Aug 18, 2019

Conversation

@nheir
Copy link
Contributor

nheir commented Nov 10, 2018

Proposal for #1552
Use same mechanism as settings from ingame menu.

@1stDecadeTW

This comment has been minimized.

Copy link

1stDecadeTW commented Nov 10, 2018

How should ingame browser look like? Maybe these features should be add:

  • Recent servers (history): it would be useful a green and grey indicator that let the user know if the server is online.
  • Friends indicator.
  • More Ideas?
@nheir

This comment has been minimized.

Copy link
Contributor Author

nheir commented Nov 10, 2018

Do this features are the same as usual server browser ?

@Sonix-

This comment has been minimized.

Copy link
Member

Sonix- commented Nov 10, 2018

If we are going to add an ingame browser then it should look the same as the main one or even simplier.

@Dune-jr

This comment has been minimized.

Copy link
Member

Dune-jr commented Nov 10, 2018

If squatting players slots is the main concern, I would propose to lock it to spectator-only. Ideally, this could come along/depend on #1580?

@nheir nheir force-pushed the nheir:feature/1552 branch 2 times, most recently from 830eb41 to 71c654b May 5, 2019
@nheir nheir force-pushed the nheir:feature/1552 branch from 71c654b to e1bf4af May 5, 2019
@fokkonaut

This comment has been minimized.

Copy link
Contributor

fokkonaut commented Aug 15, 2019

Would be cool to see this one, very usefull and nice to have. Doesnt harm anyone

@oy oy merged commit 2e43a23 into teeworlds:master Aug 18, 2019
3 checks passed
3 checks passed
ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@oy oy mentioned this pull request Aug 18, 2019
@nheir nheir deleted the nheir:feature/1552 branch Sep 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.