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

NUIPreprocessor added #356

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

NUIPreprocessor added #356

wants to merge 2 commits into from

Conversation

imihaly
Copy link

@imihaly imihaly commented Jul 5, 2016

Stylesheets are preprocessed before fed into tokenizer.
The preprocessing means:

  • dropping comments (block and single line comments are handled)
  • handling imports: @import ‘my.file.nss’
    The autoupdate feature is modified too, in order to monitor the main file and it's dependencies too.

The main goal was to get able splitting my stylesheets into smaller ones,
like collecting the colours used into one file, buttons into an other and so on.

Drawbacks:

  • the preprocessor uses NSRegularExpression to check the input which
    could be slow if intensively used, however in this case we are checking
    for 3 simple patterns.
  • the location-related token properties got broken due the imports,
    however they aren’t used on NUI-level at all.

Mihaly Imre added 2 commits July 5, 2016 13:59
Stylesheets are preprocessed before fed into tokenizer.
The preprocessing means:
- dropping comments, block and single line comments are handles
- handling imports: @import ‘my.file.nss’

Drawbacks:
- the preprocessor uses NSRegularExpression to check the input which
could be slow if intensively used, however in this case we are checking
for 3 simple patterns.
- the location-related token properties got broken due the imports,
however they aren’t used on NUI-level at all.
@vasarhelyia
Copy link

Hey guys,

Can someone take a look at this? Import would be super useful.

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.

2 participants