Skip to content

Add an adventure text colour parser#59

Merged
Citymonstret merged 1 commit into
1.1.0-devfrom
features/1.1.0/colours
Oct 16, 2020
Merged

Add an adventure text colour parser#59
Citymonstret merged 1 commit into
1.1.0-devfrom
features/1.1.0/colours

Conversation

@Citymonstret
Copy link
Copy Markdown
Member

Untested Should probably be tested before merging.

@Citymonstret Citymonstret added enhancement New feature or request parser labels Oct 16, 2020
Copy link
Copy Markdown
Member

@broccolai broccolai left a comment

Choose a reason for hiding this comment

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

Suggestions still appear red when going onto the next one

@Citymonstret Citymonstret force-pushed the features/1.1.0/colours branch from a864066 to cd00c2a Compare October 16, 2020 19:45
@jpenilla
Copy link
Copy Markdown
Member

Maybe we could have multiple modes for the parser.
One mode similar to how it is now, and a separate brig-friendly mode which will only suggest NamedTextColors but will accept a NamedTextColor or a 6-char hex.

@Citymonstret Citymonstret force-pushed the features/1.1.0/colours branch from cd00c2a to 6469bb3 Compare October 16, 2020 20:23
@Citymonstret Citymonstret force-pushed the features/1.1.0/colours branch from 6469bb3 to c8a27e2 Compare October 16, 2020 20:25
@jpenilla jpenilla self-requested a review October 16, 2020 20:27
@Citymonstret Citymonstret merged commit 9b0883e into 1.1.0-dev Oct 16, 2020
@Citymonstret Citymonstret deleted the features/1.1.0/colours branch October 16, 2020 20:35
@jpenilla jpenilla removed the request for review from broccolai October 16, 2020 20:35
@Citymonstret Citymonstret added this to the 1.1.0 milestone Jan 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request parser

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants