Skip to content
This repository has been archived by the owner on Jul 16, 2022. It is now read-only.

Better demos: the complete kitchen sink #86

Open
5 of 25 tasks
EHLOVader opened this issue Aug 4, 2015 · 9 comments
Open
5 of 25 tasks

Better demos: the complete kitchen sink #86

EHLOVader opened this issue Aug 4, 2015 · 9 comments

Comments

@EHLOVader
Copy link

EHLOVader commented Aug 4, 2015

Right now the demos are missing a few key examples, which would be helpful for testing that code doesn't break all possible uses in the future.

So I propose a better kitchen sink.

Obviously I missed many permutations but I was trying to think if all possible fringe cases where things could go wrong.

@nazar-pc
Copy link
Owner

nazar-pc commented Aug 4, 2015

I'm not sure that demo should be responsible for checking whether something is broken. Automated tests should do this, but there are no tests here yet.

@EHLOVader
Copy link
Author

Absolutely.

I agree that tests should be responsible for most of this, but when in the absence of tests I feel that a reasonable set of examples which should always work would make sense for the ease of manual testing before submitting a PR.

We have a set of examples, which fits a small portion of the use cases possible from the configuration, I just think that until there is testing it would be simple to expand the examples for this purpose.

@nazar-pc
Copy link
Owner

nazar-pc commented Aug 4, 2015

Agree, this can be done.
BTW, it would be nice to show examples alongside with code that generates them, like here: http://getuikit.com/docs/tab.html

@EHLOVader
Copy link
Author

👍 definitely.

As I understand it the main idea of PMU is that it is framework agnostic, would you suggest adding one for layout and code formatting within the demo? UIKit or Bootstrap.

@nazar-pc
Copy link
Owner

nazar-pc commented Aug 4, 2015

For simplicity it can be done without any framework, there is really not that much markup (or include some from CDN, just do not include it into repository, from those two I personally prefer UIkit), for syntax highlighting Prism is a very good one (though, it seems, they do not have any CDN).

@EHLOVader
Copy link
Author

Yeah, no need to include the framework in the repo for the demo, but having it may make it look nice.

I am sure there is a reliable third party JS cdn that has prism. Quick search turned up this... http://cdnjs.com/libraries/prism

And there is also always http://rawgit.com/

@nazar-pc
Copy link
Owner

nazar-pc commented Aug 4, 2015

I have nothing against using whatever framework in demo.

@ambicoder
Copy link

more demos...are required for sure...for example...how am I going to get the date selected as a string.
I tried to use eventListener...I get error though in the console.

@nazar-pc
Copy link
Owner

nazar-pc commented Jan 7, 2020

@ambicoder if you have difficulties, create a separate issue and add more details.
Contributions with more/better demos are definitely welcome!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants