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

296 Characters (preserves UI, margins, sizing and placeholder text) #13

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

Conversation

sidnz
Copy link

@sidnz sidnz commented Aug 15, 2018

296 characters:

Demo: https://github.com/umpox/TinyEditor/blob/b0694b20467aea032b4d99fb84ef855f014f48d5/README.md

I'm not a fan of removing placeholder text, the goal is to still be clear to the end user what each input box does, therefore the code has mostly be minimised while mostly preserving UI.

  • Using regex to generate repetitive HTML
  • Use of template literals
  • Using custom single character HTML selectors
  • Use of [id] selector and * wildcard selector
  • Use of document.write()
  • Generate single character ID from as first character from placeholder text

UI Changes:

  • font-size: 18 removed, however everything else remains.

@sidnz sidnz changed the title 327 chars while preserving UI 319 Characters - Preserving UI to be identical to original Aug 15, 2018
@sidnz sidnz changed the title 319 Characters - Preserving UI to be identical to original 305 Characters (smallest pull yet) Aug 15, 2018
@sidnz sidnz changed the title 305 Characters (smallest pull yet) 301 Characters (smallest pull yet) Aug 15, 2018
@sidnz sidnz changed the title 301 Characters (smallest pull yet) 301 Characters (preserves UI, margins, sizing and placeholder text) Aug 15, 2018
@sidnz sidnz changed the title 301 Characters (preserves UI, margins, sizing and placeholder text) 296 Characters (preserves UI, margins, sizing and placeholder text) Aug 16, 2018
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.

1 participant