Skip to content
This repository was archived by the owner on May 24, 2020. It is now read-only.

Conversation

@dev-id
Copy link
Contributor

@dev-id dev-id commented Jun 3, 2016

No description provided.

d.div({}, `Format: ${App.state.format}`),
LBox('bots', 'bots'),
LBox('timer', 'timer'),
d.div({}, d.input({
Copy link
Owner

Choose a reason for hiding this comment

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

There is a num-lands CSS class that shortens the textbox width. It might be a good idea to refactor that into a general-purpose number text box class and use it here, since the textbox is rather wide.

arxanas added a commit that referenced this pull request Jun 6, 2016
arxanas added a commit that referenced this pull request Jun 6, 2016
arxanas added a commit that referenced this pull request Jun 6, 2016
arxanas and others added 15 commits June 5, 2016 23:35
This adds a configuration file for both client and server. Page sections
(MOTD, footer) are now rendered on the client-side, which means no
orchestrating the dangerous rendering of HTML sent by the server.

You will need to generate `config.client.js` and `config.server.js`
before reloading the server.
Occasionally there is a crash because sock.h is undefined when trying
to mark a player as disconnected. It's unclear to me why this is
happening, but theoretically this commit should prevent this situation
from happening as we should now always have a `.h` on hand.
This adds a configuration file for both client and server. Page sections
(MOTD, footer) are now rendered on the client-side, which means no
orchestrating the dangerous rendering of HTML sent by the server.

You will need to generate `config.client.js` and `config.server.js`
before reloading the server.
@arxanas
Copy link
Owner

arxanas commented Jun 9, 2016

Merged the pick timer into master.

@arxanas arxanas closed this Jun 9, 2016
arxanas pushed a commit that referenced this pull request Jul 17, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants