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

Test branch #4

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Test branch #4

wants to merge 13 commits into from

Conversation

earlAchromatic
Copy link
Owner

@earlAchromatic earlAchromatic commented May 10, 2022

This change is Reviewable

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Add a Comment here

Reviewable status: 0 of 6 files reviewed, all discussions resolved

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Reviewed 6 of 6 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion


.eleventy.js line 16 at r1 (raw file):

module.exports = function (eleventyConfig) {
  eleventyConfig.addPassthroughCopy({ public: '/' });

sdgd

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

sf

Reviewed all commit messages.
Reviewable status: all files reviewed, 3 unresolved discussions


.eleventy.js line 15 at r1 (raw file):

};

module.exports = function (eleventyConfig) {

eleventyConfig.addPassthroughCopy({ public: '/' });
// eleventyConfig.addPassthroughCopy('components');


.eleventy.js line 36 at r1 (raw file):

  });

  return {

return {
dir: {


src/page.html line 11 at r1 (raw file):

      <p>Some text here</p>
      <myHeader :prop="{{prop}}">{{Header}}</myHeader>
    </div>

sdfsdf

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

To do is done

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

lorep ispum

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Some clever comment here

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

here comment is ready

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

See if this flashes here

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

See if it flashes now

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

How about now?

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

WIth overflow set to none?

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

How about scrolled down the page?

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Test it now

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

On large screen

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Keep adding comments

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Keep adding comments

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Keep adding comments

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

adding comments

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

why does it scroll

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

why does it scroll?

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Why does it scrolL?

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

asdf

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

asdf

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

asdf

Reviewable status: all files reviewed, 3 unresolved discussions

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

asdf

Reviewable status: all files reviewed, 3 unresolved discussions

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 11 at r3 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

dsf

sg

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 11 at r3 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

sg

dg

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 11 at r3 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

dg

hd

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 11 at r3 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

hd

hs

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 11 at r3 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

hs

adsg

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 11 at r3 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

adsg

dsh

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 11 at r3 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

dsh

ahh

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 9 at r3 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

gaas

ah

@earlAchromatic
Copy link
Owner Author

-- commits line 31 at r8:

Previously, earlAchromatic (Jacob Milhorn) wrote…

sf

sdfsdfsd

Code quote (i):

New commits in r7 on 7/25/2022 at 10:39 PM:

Code quote (ii):

New commits in r7 on 7/25/2022 at 10:39 PM:

Code quote (iii):

New commits in r7 on 7/25/2022 at 10:39 PM:

Code quote (iv):

New commits in r7 on 7/25/2022 at 10:39 PM:

Code quote (v):

New commits in r7 on 7/25/2022 at 10:39 PM:

Code quote (vi):

New commits in r7 on 7/25/2022 at 10:39 PM:

Code quote (vii):

New commits in r7 on 7/25/2022 at 10:39 PM:

Code quote (viii):

New commits in r7 on 7/25/2022 at 10:39 PM:

Code quote (ix):

New commits in r7 on 7/25/2022 at 10:39 PM:

Code quote (x):

New commits in r7 on 7/25/2022 at 10:39 PM:

Code quote (xi):

New commits in r7 on 7/25/2022 at 10:39 PM:

Code quote (xii):

New commits in r7 on 7/25/2022 at 10:39 PM:

@earlAchromatic
Copy link
Owner Author

-- commits line 22 at r8:
dafg

@earlAchromatic
Copy link
Owner Author

-- commits line 22 at r8:
dfs

Code quote:

- 99d707d: refactor: move index and page to src dir

- fdbb894: review changes

- d8280e7: fix: minor edits

New commits in r2 on 6/6/2022 at 8:23 PM:
- f9fc52a: no eol test A

New commits in r3 on 6/7/2022 at 10:09 PM:
- cbc4776: long line

New commits in r4 on 7/12/2022 at 2:05 PM:
- eef1fdc: add end of file newline in footer

New commits in r5 on 7/14/2022 at 5:19 PM:

@earlAchromatic
Copy link
Owner Author

Previously, earlAchromatic (Jacob Milhorn) wrote…

kh

sag

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 6 at r8 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

sdf

dfs

Suggestion (i):

const util = require(' futil');

Code quote (ii):

const util = require('futil');

Suggestion (iii):

const util = require('futconst jsdom = require('jsdom');

const util = require('util');
const util = require('futil');
il');const util = require('fuconst viteConfig = require('./vite.config.js');
const jsdom = require('jsdom');
const { JSDOM } = jsdom;
const util = require('util');
til');

Code quote (iv):

const util = require('futil');

Code quote (v):

const util = require('futil');

Code quote (vi):

const util = require('futil');

Code quote (vii):

const viteConfig = require('./vite.config.js');
const jsdom = require('jsdom');

const util = require('util');
const util = require('futil');

@earlAchromatic
Copy link
Owner Author

vite.config.js line 8 at r5 (raw file):

  plugins: [vue()],
  clearScreen: false,
  server: {

Suggestion:

  server: {sadgkljsdfklhklhflkjsalkfjklasjdfkljaslkjflkjalskdjflkjalskjdlkfjalskjdflkjsaldfjlkasjdflkjsalkdfjlkasjdflkjaslkdfjklasjdflkjaslkdjfklasjdflkjsalkdjflkajsdflkjsalkjdf

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 6 at r8 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

dfs

sgsdfsfd

fasd

sagd

sg

sdf

sf

sg

sdfsdf

sdf sdf

sfs

a
sg

sg

sf

Suggestion (i):

const EleventyVitePlugin = require('@11ty/eleventy-plugin-vite');
const viteConfig = require('./vite.config.js');
const jsdom = requisdfre('jsdom');


const util = require('util');
const util = require('fsutil');

Code snippet (ii):

ssggsdgf

Code snippet (iii):

  sdgasdlkasjdfklasjdlkfjaskldfjklasjdfklajsdklfjaklsjdflkasjdklfjaklsjdfklajskdlfjaslkjdfklasjdklfjaskljdflkajsdlkfjaskldjflksajdlfjasldkf

@earlAchromatic
Copy link
Owner Author

@oscl8tr

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 33 at r3 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

fx

code

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 16 at r4 (raw file):

Heading 1

Heading 2

Heading 3

Heading 4

Heading 5

Heading 6

  • Bullet point
    • Sub-bullet point
      • Sub-sub-bullet point
  1. Ordered list item
  2. Another ordered list item
  3. Yet another ordered list item

This is a paragraph.

This is a blockquote.

This is inline code.

This is a code block.

Column 1 Column 2
Row 1, Column 1 Row 1, Column 2
Row 2, Column 1 Row 2, Column 2

Bold text

Italicized text

Strikethrough text

Link text

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 17 at r5 (raw file):

eleventyConfig) {eleventyConfig.addPassthroughCopy('components');eleventyConfig.addPassthroughCopy('node_meleventyConfig.addPassthroughCopy('components');eleventyConfig.addPassthroughCopy('node_m

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 16 at r4 (raw file):

Heading 1

Heading 2

Heading 3

Heading 4

Heading 5

Heading 6

  • Bullet point
    • Sub-bullet point
      • Sub-sub-bullet point
  1. Ordered list item
  2. Another ordered list item
  3. Yet another ordered list item

This is a paragraph.

This is a blockquote.

This is inline code.

This is a code block.

Column 1 Column 2
Row 1, Column 1 Row 1, Column 2
Row 2, Column 1 Row 2, Column 2

Bold text

Italicized text

Strikethrough text

Link text

@earlAchromatic
Copy link
Owner Author

.eleventy.js line 18 at r5 (raw file):

sf

@earlAchromatic
Copy link
Owner Author

:lgtm_cancel:

@earlAchromatic
Copy link
Owner Author

:lgtm:

Copy link
Collaborator

@oscl8tr oscl8tr left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 4 files at r5.
Reviewable status: 9 of 12 files reviewed, 37 unresolved discussions

Copy link
Collaborator

@oscl8tr oscl8tr left a comment

Choose a reason for hiding this comment

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

hlk

Reviewable status: 9 of 12 files reviewed, 37 unresolved discussions

@earlAchromatic
Copy link
Owner Author

-- commits line 20 at r8:
sgdf

Code quote:

New commits in r4 on 7/12/2022 at 2:05 PM:
- eef1fdc: add end of file newline in footer

@earlAchromatic
Copy link
Owner Author

-- commits line 22 at r8:

Previously, earlAchromatic (Jacob Milhorn) wrote…

dafg


:lgtm:

Code snippet (i):

<time/> is a mystery </ss?>

<div> 

Code quote (ii):

New commits in r5 on 7/14/2022 at 5:19 PM:

@earlAchromatic
Copy link
Owner Author

-- commits line 22 at r8:


New commits in r2 on 6/6/2022 at 8:23 PM:
- f9fc52a: no eol test A

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

https://reviewable.io/lgtm.png

https://reviewable.io/lgtm.png

https://reviewable.io/lgtm.png

https://reviewable.io/lgtm.png

Reviewed 1 of 6 files at r1, all commit messages.
Reviewable status: 10 of 12 files reviewed, 60 unresolved discussions (waiting on @oscl8tr)


a discussion (no related file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

sag

:lgtm:

___ *[`-- commits` line 24 at r8](http://localhost:8080/reviews/earlAchromatic/11ty-vue-pragmatic-components/4#-N9m86Vt3DBLIl370paV:-NUPLdLB9ImeCBzR9BAn:b-sqyskd):* > ``` > b287869: remove some things to test elid > ```
- b287869: remove some things to test elide
emove some t

-- commits line 31 at r8:

sdfsdfsd

mits in r7 on 7/25/20

.eleventy.js line 17 at r3 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

sdf

sdf

Code quote:

    async ({ dir, results, runMode, outputMode }) => {

.eleventy.js line 19 at r3 (raw file):

// This is some text t//

.eleventy.js line 25 at r3 (raw file):

Previously, earlAchromatic (Jacob Milhorn) wrote…

dfds

dsf

Code snippet:

sdfsdf

.eleventy.js line 33 at r3 (raw file):

code

.eleventy.js line 11 at r5 (raw file):

nts/myHeader.vue',

  myFooter: '../components/myFoote
myHeader: '../components/myHeader.vue',

  myFooter: '../components/myFooter.

.eleventy.js line 17 at r5 (raw file):

  _findOppositeSchemes(scheme) {
    if (_.has(EXCLUSIVE_COLOR_SCHEME_MAP, scheme)) {
      const result = EXCLUSIVE_COLOR_SCHEME_MAP[scheme];
      return _.isArray(result) ? result : [result];
    }
    const result = _.findKey(EXCLUSIVE_COLOR_SCHEME_MAP, colorScheme => {
      return _.isArray(colorScheme) ? _.includes(colorScheme, scheme) : colorScheme === scheme;
    });
    return result ? [result] : [];
  }sd
> yConfig) {eleventyConfig.a
> ```

# Heading 1

## Heading 2

### Heading 3

### Heading 4

### Heading 5

### Heading 6

.eleventy.js line 17 at r6 (raw file):

  eleventyConfig.addPassthroughCopy('components');

  eleventyConfig.addPassthroughCopy('node_modules');

.eleventy.js line 21 at r6 (raw file):

y.after',
    async ({ d
    'eleventy.asfter',

    async ({ dir, ressults, runMode, outputMode }) => {

.eleventy.js line 6 at r8 (raw file):

dfs

const util = require(' futil');

.eleventy.js line 11 at r8 (raw file):

dfs


vite.config.js line 6 at r5 (raw file):

efault defineConfig({


  plugins

vite.config.js line 16 at r6 (raw file):

solve: {
},

build: {
mode: 'production',


-- commits line 22 at r8:
sdfdssdfsdf


src/page.html line 5 at r8 (raw file):

prop: '125'
---

My thought is that what if you did it differently?


.eleventy.js line 1 at r2 (raw file):

const EleventyVitePlugin = require('@11ty/eleventy-plugin-vite');

console.log(markdown, ev.clipboardData.getData('text/markdown'));


.eleventy.js line 4 at r2 (raw file):

const viteConfig = require('./vite.config.js');
const jsdom = require('jsdom');
const { JSDOM } = jsdom;

sdf


.eleventy.js line 60 at r2 (raw file):

    if (!comp) {
      continue;
    }

sdf


.eleventy.js line 93 at r2 (raw file):

    el.setAttribute('injectionPoint', true);
    el.innerHTML = VueWrapper;
    comp.replaceWith(el);

sdfsdf


.eleventy.js line 95 at r2 (raw file):

    comp.replaceWith(el);
  }
}

sdfsdf


.eleventy.js line 33 at r3 (raw file):

    dir: {
      input: 'src',
    },

sd


.eleventy.js line 43 at r3 (raw file):

  tryComponents(doc);

  return dom.serialize();

sf


.eleventy.js line 10 at r4 (raw file):

const componentRegistry = {
  myHeader: '../components/myHeader.vue',

sdf


.eleventy.js line 64 at r4 (raw file):

    //check comp recursively against component registry

    //let props = Object.values(comp.attributes);

sdf


.eleventy.js line 70 at r4 (raw file):

<script type=module>
import {createApp} from "../node_modules/vue/dist/vue.esm-bundler.js";
import ${registeredComponent} from "${registeredComponentPath}";

sdf


.eleventy.js line 85 at r4 (raw file):

`;
    let el = doc.createElement('div');
    el.setAttribute('injectionPoint', true);

sdf


.eleventy.js line 88 at r4 (raw file):

    el.innerHTML = VueWrapper;
    comp.replaceWith(el);
  }

sdf


.eleventy.js line 89 at r4 (raw file):

    comp.replaceWith(el);
  }
}

sdfsdf


.eleventy.js line 13 at r8 (raw file):

  myHeader: '../components/myHeader.vue',
  myFooter: '../components/myFooter.vue',
  Loading: '../components/Loading.vue'

sdfHeading 1


.eleventy.js line 52 at r8 (raw file):

    let componentString = '';

    let tempComponentReg = [];

s


.eleventy.js line 70 at r8 (raw file):

  template: \`
  <${registeredComponent}>
  ${childTemplate}

sdf


.eleventy.js line 88 at r2 (raw file):

    comp.replaceWith(el);
  }
}

sdfsdf


.eleventy.js line 88 at r2 (raw file):

    comp.replaceWith(el);
  }
}

sdf


.eleventy.js line 88 at r2 (raw file):

    comp.replaceWith(el);
  }
}

dsf


.eleventy.js line 88 at r2 (raw file):

    comp.replaceWith(el);
  }
}

sdfsdf


.eleventy.js line 88 at r2 (raw file):

    comp.replaceWith(el);
  }
}

sdf


.eleventy.js line 15 at r3 (raw file):

module.exports = function (eleventyConfig) {
  eleventyConfig.addPassthroughCopy({ public: '/' });

sagsdf## Heading 2

Heading 3

Heading 4

Heading 5

Code quote:

module.exports = function (eleventyConfig) {
  eleventyConfig.addPassthroughCopy({ public: '/' });

.eleventy.js line 18 at r5 (raw file):

  eleventyConfig.addPassthroughCopy('components');
  eleventyConfig.addPassthroughCopy('node_modules');

sdf


.eleventy.js line 18 at r5 (raw file):

  eleventyConfig.addPassthroughCopy('components');
  eleventyConfig.addPassthroughCopy('node_modules');

sdfsdf

const a = {my: "string"}

Code snippet:

dfg

.eleventy.js line 20 at r8 (raw file):

  eleventyConfig.on(
    'eleventy.after',

sdf

Code quote:

module.exports = function (eleventyConfig) {

.eleventy.js line 18 at r5 (raw file):

module.exports = function (eleventyConfig) {
  eleventyConfig.addPassthroughCopy({ public: '/' }); eleventyConfig.addPassthroughCopy('components');eleventyConfig.addPassthroughCopy('node_modules');eleventyConfig.on('eleventy.after',
    async ({ dir, results, runMode, outputMode }) => {

sdf


.eleventy.js line 57 at r5 (raw file):

    }

    let childTemplate = comp.innerHTML;

sd

Copy link
Owner Author

@earlAchromatic earlAchromatic left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 6 files at r1, all commit messages.
Reviewable status: 10 of 12 files reviewed, 60 unresolved discussions (waiting on @oscl8tr)

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