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

Using initial design by James, created a Vue3 template starter. #140

Merged
merged 65 commits into from
Jun 4, 2023
Merged

Using initial design by James, created a Vue3 template starter. #140

merged 65 commits into from
Jun 4, 2023

Conversation

kinxiel
Copy link
Collaborator

@kinxiel kinxiel commented Oct 8, 2022

Design

You can find the design for the MVP, care of Martin here.

Description, Motivation, Context

  • Project uses Vue/Nuxt3 for the frontend and Vite as the build tool. Planning to add Vitest and Cypress for unit testing and e2e testing later on.
  • Project uses TypeScript with Vue but compilation is not set to strict, so you can have type errors and it will still run.
  • Data is generated by a Node script. Both the data and the scripts can be found here
  • Page should be responsive. The design for both the desktop and mobile views can be found in the design.
  • Used to use pnpm but because of some problems, switched to normal npm.

Todos

General

  • CSS: Responsive Media Queries for each breakpoint. It's in the design but hasn't been implemented yet.

Desktop

  • Home Page
  • Menu
  • Photos Page.
  • Photos Page: how to handle videos (new issue)
  • About Page.
  • Join Page.
  • Events Page.
  • Single Event Page. Page to view or RSVP for a single event. Since Meetup does not allow access to past events, should probably come up with a separate page on the site for past events. Perhaps something Markdown based.
  • 404 page.

Mobile

  • Home: Minor Adjustments
  • Menu: Hamburger menu, mobile view, etc.
  • Photos Page: works ok I guess.
  • About Page.
  • Join Page.
  • Events Page. It's usable but needs to match the design.
  • Single Event Page.
  • 404 page. Haven't checked on mobile.
  • Responsive adjustments for iPhone notch area.

Tests

  • Unit tests (Vitest)
  • e2e tests (Cypress)

Todo: Probably not for MVP but later

  • Survey results visualization
  • Dark mode (probably a nice to have but not necessary at the moment)

Notes

To accommodate both the desktop and mobile versions, some components need to be updated, feel free to do so.

Copy link
Member

@martinheidegger martinheidegger left a comment

Choose a reason for hiding this comment

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

Hello @kinxiel 👋!

This has already been a bunch of nice work! 🤩 Looking over it, though, I have found quite a few tasks... 😅

  • Link the Subscribe button to the subscription service.

    This is how it currently works.

  • For the deploy to work, the build script needs to be adjusted, right?

  • "What We Do"

    We are a independent group of designers and developers in the Kansai (Osaka/Kyoto) area of Japan. To practice our English presentations skills, learns new things and meet other people we are organizing IRL (in-person) events.
    All our events are organized by volunteers. Any money that we collect is just to cover our costs.

  • Photos Page We can use our Flickr Group for the pictures. It even has an API. Maybe @Kimmym6a has an idea for how to display them? Until then maybe a link is enough?

  • Contact Page Yes. This ties into another isse about SNS use of owddm. Will bring it up in an other issue. (more on that later)

  • Page for each event. That'd be awesome!

@@ -1,44 +1,42 @@
{
"name": "owddm",
"version": "1.0.0",
Copy link
Member

Choose a reason for hiding this comment

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

Lot's here is better to be kept, particularly the "private": true flag.

src/components/SiteMainNewsletter.vue Outdated Show resolved Hide resolved
src/components/__tests__/HelloWorld.spec.ts Outdated Show resolved Hide resolved
index-copy.html Outdated Show resolved Hide resolved
@@ -1,3 +0,0 @@
{{- if not .Site.IsServer -}}
Copy link
Member

Choose a reason for hiding this comment

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

Can we re-add this?

"htmlhint": "htmlhint docs/_site/**/*.html",
"serve": "npx hugo serve",
"spellcheck": "npx cspell \"content/**/*.md\"",
"svgo": "npm-run-all --parallel \"svgo:base -- --folder=docs/img/header\"",
Copy link
Member

Choose a reason for hiding this comment

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

Is it possible to keep the useful scripts in here? spellcheck, svgo, css-lint and html-hint seem to be a good idea?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Do we mix Vue with Hugo?

Copy link
Member

Choose a reason for hiding this comment

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

No, the hugo scripts make little sense, but the linting may be a good idea. Spellcheck, linting etc.

pluralizeListTitles: false
googleAnalytics: "disabled"

params:
Copy link
Member

Choose a reason for hiding this comment

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

This set's a bunch of meta header data that seems missing from the vue template?

Here is the rendered header.

<meta property="og:description" content="Osaka Web Designers and Developers Meetup">
<meta property="og:type" content="website">
<meta property="og:url" content="https://owddm.com/">
<meta property="og:site_name" content="owddm">
<meta name=twitter:card content="summary">
<meta name=twitter:site content="@owddm">
<meta name=theme-color content="#005f9e">
<meta name=geo.placename content="Osaka, jp">
<meta name=geo.region content="jp">

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

isn't this config.yml part of the Hugo config?

Copy link
Member

Choose a reason for hiding this comment

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

It is, but the header makes sense on the regular site as well, right?

docs/index.html Show resolved Hide resolved
@@ -1,30 +0,0 @@
{{ define "main" }}

Copy link
Member

Choose a reason for hiding this comment

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

The 404.html has not been set up. But if we had one 404s would work. This would even fix #12

static/CNAME Outdated Show resolved Hide resolved
@martinheidegger
Copy link
Member

@kinxiel I added the FLICKR_SECRET and FLICKR_KEY as secrets to the organization and as long as its not a fork, I think you should be able to use it in github actions.

@kinxiel
Copy link
Collaborator Author

kinxiel commented Nov 5, 2022

Thanks. I'll try to see if we can make some kind of gallery page.

@kinxiel
Copy link
Collaborator Author

kinxiel commented Dec 9, 2022

Migrated to Nuxt 3 from Vue for easier routing and etc.

@martinheidegger
Copy link
Member

@kinxiel Awesome I think its almost there as well! 😻 There is quite a few things that can be better, A few things have priority in my book:

  • Remove links that point nowhere (RSS/etc.) and add issues to add each of them.
  • Fix the rendering of the meetup text to break lines properly
  • Make the KWDDM header actually say KWDDM 😓
  • Make sure that the links outside open new windows
  • Let the github icon point to the homepage repository
  • Create issues for all the things need to be fixed.

@kinxiel
Copy link
Collaborator Author

kinxiel commented May 11, 2023

  • Remove links that point nowhere (RSS/etc.) and add issues to add each of them.
  • Fix the rendering of the meetup text to break lines properly
  • Make the KWDDM header actually say KWDDM 😓
  • Make sure that the links outside open new windows
  • Let the github icon point to the homepage repository
  • Create issues for all the things need to be fixed.

@martinheidegger
Copy link
Member

With the additional things in https://github.com/kinxiel/owddm.com/pull/6 we are mostly at a new version.

Thought one thing we still kind of need → <head><title>... - without that its a bit ... sad.

Other than that, I think the rest is "next step"

kinxiel and others added 23 commits June 4, 2023 14:00
…ated various parts of the design into their own separate components.
kinxiel and others added 25 commits June 4, 2023 14:04
… out the survey menu for now, since it is not yet ready.
…d be a link semantically. Use NuxtLink activeClass prop where applicable. Other minor bug fixes.
…th mobile and regular versions with consistent spacing of the banner to the var(--space) definition.
@martinheidegger
Copy link
Member

@kinxiel I rebased this branch at https://github.com/owddm/owddm.com/tree/kmain - you should be able to use it with

git fetch && git reset --hard origin/kmain

and then git push -f or comparable.

Copy link
Member

@martinheidegger martinheidegger left a comment

Choose a reason for hiding this comment

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

Good first version of the new homepage I think

@martinheidegger martinheidegger merged commit 74b4ff9 into owddm:main Jun 4, 2023
@martinheidegger martinheidegger mentioned this pull request Jun 4, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants