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

[material-ui][docs] Make the free premium templates easier to use #14604

Open
askpatrickw opened this issue Feb 21, 2019 · 16 comments
Open

[material-ui][docs] Make the free premium templates easier to use #14604

askpatrickw opened this issue Feb 21, 2019 · 16 comments
Labels
docs Improvements or additions to the documentation ready to take Help wanted. Guidance available. There is a high chance the change will be accepted

Comments

@askpatrickw
Copy link

I'm looking through the Onepirate landing page example as a launching point and I'm not understanding how the links in AppAppBar and AppFooter are working.

The Source for the onepirate example is here:
https://github.com/mui-org/material-ui/tree/master/docs/src/pages/premium-themes/onepirate

Expected:
Download the example and have it work.

Observed:

  • 404 to Terms, Privacy, Sign-In, etc...
  • Many other files not within the example folder such as (all images, and some other components.)

I've gathered up the files not in the source directory, but I'm looking for a straight-forward way to fix the 404s.

For example this link to the terms in ./modules/views/AppFooter.js

<li className={classes.listItem}>
    <Link href="/premium-themes/onepirate/terms">Terms</Link>
</li>

You will note two things:

  1. The link is to /premium-themes/ not to files in the source directory linked to from the Premium Themes library
  2. There is not router in the example

I've tried moving the Terms.js from my src into into /src/modules and putting the terms.js into src and
changed the href to '/terms' but I still get 404s.

It seems as though this example is not really reusable as-is, but I'm not sure. Do I need to re-work the example using react-router-dom or is there some other more straight-forward approach I'm not seeing.

I'm dropping this into a vanilla create-react-app project and serving it up with AWS Amplify hosting.

Module Versions: 
+-- @material-ui/core@3.9.2
+-- @material-ui/icons@3.0.2
+-- aws-amplify@1.1.19
+-- aws-amplify-react@2.3.0
+-- clsx@1.0.3
+-- prop-types@15.7.2
+-- react@16.8.2
+-- react-dom@16.8.2
+-- react-router-dom@4.3.1
`-- react-scripts@2.1.5
@askpatrickw
Copy link
Author

@oliviertassinari I see you created the onepirate example. Can you suggest a path forward? I'd be happy to provide additional documentation for the next person who wants to use the example. tx!

@askpatrickw
Copy link
Author

I decided to use material-kit instead.

@joshwooding
Copy link
Member

Links are a wrapper around an anchor element that applies Material UI styling. The router we use is built in to Next (next/router)

@askpatrickw
Copy link
Author

Ah.. so there are other dependencies needed to make this work, but there's no package.json specific to the example.

My $.02 is these examples should be in their own repos and work independently not be based on the documentation app.

Also, there's a general comment in the docs that you can drop the themes into the sample applications, but again there are dependencies which would not be specified in the theme or the sample.

I hope that's helpful feedback.

@oliviertassinari
Copy link
Member

oliviertassinari commented Feb 24, 2019

@askpatrickw Thank you for raising the problem. You are right, we should make our "theme" better isolated!

@oliviertassinari oliviertassinari added docs Improvements or additions to the documentation priority: important This change can make a difference labels Feb 24, 2019
@askpatrickw
Copy link
Author

I suggest you move themes to their own repro with folders for each theme, and you can link to other repos as well.

See https://github.com/getpelican/pelican-themes

@kpennell
Copy link

kpennell commented Mar 2, 2019

@oliviertassinari Thank you for working on this. Would love to have solid theme starting points

@neilpalima
Copy link

neilpalima commented Jun 13, 2019

Is this done? I encounter this problem as well. I would love to use this with create-react-app.

@oliviertassinari oliviertassinari changed the title Question about links in Onepirate Premium Theme Sample Make the full theme examples easier to use Jul 4, 2019
@oliviertassinari oliviertassinari removed the priority: important This change can make a difference label Dec 1, 2019
@gregsifr
Copy link

gregsifr commented Apr 5, 2020

As a beginner I would also appreciate the theme working out of the box

@Nived660987

This comment has been minimized.

@ehartford
Copy link

I have same problem, as an experienced web developer learning material-ui, I would like to have a starting point to build from that works out of the box with no modification. This one, I had to track down all the static files, and still have problems with getting the markdown to generate with webpack.

@tobiaspfuetze
Copy link

Since this seems still to be an open issue, I created a version with create-react-app and added 'react-router-dom' as one of the dependencies. All credit to @oliviertassinari who created the template in the first place. Feel free to git clone https://github.com/tobiaspfuetze/material-ui-onepirate.git && cd material-ui-onepirate && npm install && npm start

@mariabohorquez
Copy link

Since this seems still to be an open issue, I created a version with create-react-app and added 'react-router-dom' as one of the dependencies. All credit to @oliviertassinari who created the template in the first place. Feel free to git clone https://github.com/tobiaspfuetze/material-ui-onepirate.git && cd material-ui-onepirate && npm install && npm start

Thank you a lot for making it a separated repo! It's a lot easier to work with. Small comment, would you happen to know why your version has all the button text underlined? Seems a style is missing somewhere.

@yukihirai0505
Copy link

FYI

I found a repository that uses the Typescript for the onepirate theme.
https://github.com/rothbart/onepirate-typescript

@kriscoleman
Copy link

FYI

I found a repository that uses the Typescript for the onepirate theme. https://github.com/rothbart/onepirate-typescript

this was actually super helpful and enough for it to click so I could start using this in a gatsby app. thanks.

@samuelsycamore samuelsycamore changed the title Make the full theme examples easier to use [material-ui][docs] Make the full theme examples easier to use Nov 28, 2023
@samuelsycamore
Copy link
Member

samuelsycamore commented Nov 28, 2023

Revisiting this in Q4 '23—it seems that these two free "premium" themes each need a package.json and a README.md to explain how to use them. They should "just work" after you npm install.

The rothbart/onepirate-typescript repo linked above is a good example, but I'd much prefer to see Vite or Next.js in place of create-react-app for these.

@samuelsycamore samuelsycamore changed the title [material-ui][docs] Make the full theme examples easier to use [material-ui][docs] Make the free premium templates easier to use Nov 28, 2023
@samuelsycamore samuelsycamore added the ready to take Help wanted. Guidance available. There is a high chance the change will be accepted label Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation ready to take Help wanted. Guidance available. There is a high chance the change will be accepted
Projects
None yet
Development

No branches or pull requests