Redesign #27

webkonstantin · 2019-01-04T18:04:02Z

#26

Еще в работе, но пока в таком виде https://radio-t.netlify.com/

в теле подкаста скрывается все, кроме первого маркированного списка, который считается списком тем. с этим надо будет что-то сделать, т. к. там бывают всякие спонсоры и произвольный контент. можно например показывать его только внутри страницы подкаста, но не показывать на главной
временные метки из списка тем вытаскиваются с помощью js
есть самописный аудио плеер. можно гулять по сайту, не прерывая воспроизведение
по нажатию на метку времени запускается плеер в нужном месте

TODO

страница и баннер онлайн вещания
баннер Пираты Друзья Помочь DigitalOcean
последние комментарии в боковой колонке
поиск по сайту
подготовить для маленьких экранов
темная тема

webkonstantin · 2019-05-12T19:03:43Z

@umputun все поправил, кроме перехода на рекламу. cannot reproduce, как говорится

@Reeywhaar спасибо, взял твой компонент, работает :)

umputun · 2019-05-12T19:16:35Z

кроме перехода на рекламу. cannot reproduce, как говорится

Да, я понял в чем дело было. Это uBlock реагирует на advertising в url. Отключение помогло

umputun · 2019-05-13T15:58:34Z

по поводу запуска. Из товего списка осталось:

плашки Друзья и Помочь рядом с баннером DigitalOcean ведут в одно и то же место

можно оставиль только одну, и назвать ее как-то обще, типа "Помочь подкасту / друзья", что-то в этом роде. Лишнюю плашку можно натравить на news.radio-t.com

стоит обновить обложку

Это можно, вто только кто бы ее нарисовал.

кнопку play для online-вещания наверно стоит скрывать, пока время вещания не наступило

звучит разумно

Khilkov · 2019-05-13T16:32:01Z

Нарисовали, даже две )))

umputun · 2019-05-13T16:41:28Z

@Khilkov прелестно. Я поменяю в фиде на ту, что справа когда переключимся на новую версию

This reverts commit 7d62310, reversing changes made to f4a933e.

webkonstantin · 2019-05-14T18:29:46Z

LGTM?

umputun · 2019-05-14T18:42:32Z

несколько моментов:

про sidebar. Вроде по дизайну он должен был скрываться/показыаваться и там вроде и чат подразумевался в отдельном табе. Это, конечно, не шоу-стоппер, можем и без этого релизить
конфликт с мастером, было бы круто избежать force при merge
Мне непонятно вот что - какие-то изменения автоматизации нужны? Вроде нет, но хочу подтверждения ;)

UPD: и еще одно - иконка на плашке про news мне непонятна, это что, руки вверх?

webkonstantin · 2019-05-14T19:04:04Z

сайдбар на больших экранах всегда показывается, а на маленьких можно показывать/скрывать. гиттер вписать красиво не получилось, пришлось убрать, он конфликтовал с плеером/сайдбаром. мне почему-то кажется что мало кто пользовался этим встроенным виджетом. ссылки на чат везде есть

конфликтов не должно быть. даже гитхаб здесь пишет This branch has no conflicts with the base branch

Dockerfile и прочие скрипты я обновил, ничего не нужно, если я ничего не упустил

я вижу high five :)

umputun · 2019-05-14T19:07:32Z

сайдбар на больших экранах всегда показывается, а на маленьких можно показывать/скрывать. гиттер вписать красиво не получилось, пришлось убрать, он конфликтовал с плеером/сайдбаром. мне почему-то кажется что мало кто пользовался этим встроенным виджетом. ссылки на чат везде есть

Ага, увидел. Хорошо, да.

конфликтов не должно быть. даже гитхаб здесь пишет This branch has no conflicts with the base branch

тоже твоя правда. он не в силах был сделать rebase, а merge согласен. Кстати, мне их squash или пусть будут оригинальными?

Dockerfile и прочие скрипты я обновил, ничего не нужно, если я ничего не упустил

счас посмотрю на них свежим взглядом

umputun · 2019-05-14T19:12:09Z

счас посмотрю на них свежим взглядом

вроде все нормально, если что то пофиксим в процессе.

Я готов вливать, дай знать как прдепочитаешь squash или без

umputun · 2019-05-14T19:23:26Z

нет, что-то поломанно. Я попробовал построить контейнер (docker-compose build). Оно, во первых упало, а во вторых "found 9 vulnerabilities (2 low, 1 moderate, 6 high)"


added 1511 packages from 1023 contributors and audited 24267 packages in 28.099s
found 9 vulnerabilities (2 low, 1 moderate, 6 high)
  run `npm audit fix` to fix them, or `npm audit` for details
Removing intermediate container 9c88961774c6
 ---> d1467f21e491
Step 7/19 : COPY hugo/.modernizr.js hugo/webpack.mix.js ./
 ---> f7c0b1497761
Step 8/19 : COPY hugo/src/ src/
 ---> f49302483d52
Step 9/19 : RUN npm run production
 ---> Running in 6dd6f60c1fdd

> @ production /app
> cross-env NODE_ENV=production node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js

 94% after seal ERROR  Failed to compile with 2 errors7:20:49 PM                  

 error  in ./src/js/components/remark.tsx

Module build failed (from ./node_modules/ts-loader/index.js):
Error: error while parsing tsconfig.json
    at Object.loader (/app/node_modules/ts-loader/dist/index.js:19:18)

 @ ./src/js/controllers/comments_embed_controller.jsx 26:0-42 46:33-39
 @ ./src/js/controllers sync \.jsx?$
 @ ./src/js/stimulus.js
 @ ./src/js/app.js
 @ multi ./src/js/app.js ./src/scss/app.scss ./src/scss/vendor.scss

 error  

[tsl] ERROR
      TS18002: The 'files' list in config file 'tsconfig.json' is empty.

 @ ./src/js/controllers/comments_embed_controller.jsx 26:0-42 46:33-39
 @ ./src/js/controllers sync \.jsx?$
 @ ./src/js/stimulus.js
 @ ./src/js/app.js
 @ multi ./src/js/app.js ./src/scss/app.scss ./src/scss/vendor.scss

Error: ENOENT: no such file or directory, open '/app/static/build/modernizr-bundle.js'
    at Object.openSync (fs.js:438:3)
    at Object.readFileSync (fs.js:343:35)
    at File.read (/app/node_modules/laravel-mix/src/File.js:166:19)
    at File.version (/app/node_modules/laravel-mix/src/File.js:173:25)
    at Manifest.hash (/app/node_modules/laravel-mix/src/Manifest.js:52:65)
    at assets.data.files.map.file (/app/node_modules/laravel-mix/src/tasks/VersionFilesTask.js:14:26)
    at Array.map (<anonymous>)
    at VersionFilesTask.run (/app/node_modules/laravel-mix/src/tasks/VersionFilesTask.js:11:39)
    at CustomTasksPlugin.runTask (/app/node_modules/laravel-mix/src/webpackPlugins/CustomTasksPlugin.js:35:14)
    at Mix.tasks.forEach.task (/app/node_modules/laravel-mix/src/webpackPlugins/CustomTasksPlugin.js:11:44)
    at Array.forEach (<anonymous>)
    at compiler.plugin.stats (/app/node_modules/laravel-mix/src/webpackPlugins/CustomTasksPlugin.js:11:23)
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/app/node_modules/tapable/lib/HookCodeFactory.js:32:10), <anonymous>:18:1)
    at AsyncSeriesHook.lazyCompileHook (/app/node_modules/tapable/lib/Hook.js:154:20)
    at onCompiled (/app/node_modules/webpack/lib/Compiler.js:234:21)
    at hooks.afterCompile.callAsync.err (/app/node_modules/webpack/lib/Compiler.js:631:15)
    at _err0 (eval at create (/app/node_modules/tapable/lib/HookCodeFactory.js:32:10), <anonymous>:11:1)
    at /app/node_modules/modernizr-webpack-plugin/index.js:157:7
    at Object.requireConfig.out (/app/node_modules/modernizr/lib/build.js:106:7)
    at /app/node_modules/requirejs/bin/r.js:28864:24
    at /app/node_modules/requirejs/bin/r.js:3041:37
    at /app/node_modules/requirejs/bin/r.js:2989:25
    at Function.prim.nextTick (/app/node_modules/requirejs/bin/r.js:28083:9)
    at Object.callback (/app/node_modules/requirejs/bin/r.js:2988:26)
    at Object.then (/app/node_modules/requirejs/bin/r.js:3038:23)
    at Function.build._run (/app/node_modules/requirejs/bin/r.js:28612:12)
    at /app/node_modules/requirejs/bin/r.js:28288:26
    at /app/node_modules/requirejs/bin/r.js:3041:37
    at /app/node_modules/requirejs/bin/r.js:2989:25
    at Function.prim.nextTick (/app/node_modules/requirejs/bin/r.js:28083:9)
    at Object.callback (/app/node_modules/requirejs/bin/r.js:2988:26)
    at Object.then (/app/node_modules/requirejs/bin/r.js:3038:23)
    at Object.start (/app/node_modules/requirejs/bin/r.js:3031:34)
    at build (/app/node_modules/requirejs/bin/r.js:28265:23)
    at runBuild (/app/node_modules/requirejs/bin/r.js:30302:17)
    at Object.execCb (/app/node_modules/requirejs/bin/r.js:1946:33)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! @ production: `cross-env NODE_ENV=production node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js`
npm ERR! Exit status 1
npm ERR! 
npm ERR! Failed at the @ production script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2019-05-14T19_20_50_118Z-debug.log
ERROR: Service 'hugo' failed to build: The command '/bin/sh -c npm run production' returned a non-zero code: 1

webkonstantin · 2019-05-14T21:51:31Z

поправил сборку

думаю можно просто смерджить без сквоша

umputun · 2019-05-14T23:09:19Z

теперь строится, но остались уязвимости:

added 1511 packages from 1023 contributors and audited 24267 packages in 45.245s
found 4 vulnerabilities (2 low, 1 moderate, 1 high)

github на такое будет ругаться всячески, хорошо бы обновить

webkonstantin · 2019-05-14T23:43:50Z

Уже обновил все что можно

Можно релизить :)

Reeywhaar · 2019-05-15T00:05:01Z

Сейчас заметил, что в ios 9.3 (максимальная для iphone 4s) все поломано: SyntaxError: Unexpected token '>' app.js:1. Там надо смотреть, сафари там не умеет es6: let, const, arrow functions etc...
В 10.3 все ок.

umputun · 2019-05-15T00:29:03Z

а вот и стрдания от github подоспели:

Known high severity security vulnerability detected in tar < 4.4.2 defined in package-lock.json.
package-lock.json update suggested: tar ~> 4.4.2.

Known high severity security vulnerability detected in pyyaml < 4.2b1 defined in requirements-redesign.txt.
requirements-redesign.txt update suggested: pyyaml ~> 4.2b1.

webkonstantin added 30 commits December 31, 2018 03:00

wip redesign

86d98d3

use base layout

819be70

wip

bc9aed2

omg redesign

91fd36a

add code highlight

db262f4

fix time labels, add online banner

ea9030e

play button

0463a31

audio player wip

016387c

audio player wip

0ef8c80

audio player wip

771710f

update build

c0ea9e7

split title with hugo regex

cf3a165

highlight go template syntax

50a5569

editorconfig

4b26b89

format layouts

98b4b65

likely

d728ab0

use node sass

4f7a038

fix

796e8a7

fix links

6df9830

fix links

0ce4b81

footer

57f3f70

single post

987e73c

todo

138d64a

delete legacy javascript

f1a3528

about page

c8b8262

fix

3642ed2

archives

2b1448c

typo

7cb6a59

typo

043d468

navbar

d4790e2

webkonstantin added 3 commits May 12, 2019 18:25

navbar

d55b516

bullet

64f73f0

fix player start from time label

eddc050

fix

d3c08a2

webkonstantin added 5 commits May 14, 2019 18:50

update covers

41278c5

banner for news

f4a933e

Merge branch 'redesign-help' into redesign

7d62310

Revert "Merge branch 'redesign-help' into redesign"

ac4ca8f

This reverts commit 7d62310, reversing changes made to f4a933e.

show online play button during showtime only

b2da5c7

fix build

e9ff864

webkonstantin added 2 commits May 15, 2019 00:27

set env to render yandex metrica and google analytics

dc4cd12

remove legacy

cfeaac2

umputun merged commit a259389 into radio-t:master May 14, 2019

webkonstantin mentioned this pull request Jul 21, 2021

поддержка webp для обложек подкастов #238

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Redesign #27

Redesign #27

webkonstantin commented Jan 4, 2019 •

edited

Loading

webkonstantin commented May 12, 2019

umputun commented May 12, 2019

umputun commented May 13, 2019

Khilkov commented May 13, 2019

umputun commented May 13, 2019 •

edited

Loading

webkonstantin commented May 14, 2019

umputun commented May 14, 2019 •

edited

Loading

webkonstantin commented May 14, 2019

umputun commented May 14, 2019

umputun commented May 14, 2019

umputun commented May 14, 2019

webkonstantin commented May 14, 2019

umputun commented May 14, 2019

webkonstantin commented May 14, 2019

Reeywhaar commented May 15, 2019 •

edited

Loading

umputun commented May 15, 2019

Redesign #27

Redesign #27

Conversation

webkonstantin commented Jan 4, 2019 • edited Loading

webkonstantin commented May 12, 2019

umputun commented May 12, 2019

umputun commented May 13, 2019

Khilkov commented May 13, 2019

umputun commented May 13, 2019 • edited Loading

webkonstantin commented May 14, 2019

umputun commented May 14, 2019 • edited Loading

webkonstantin commented May 14, 2019

umputun commented May 14, 2019

umputun commented May 14, 2019

umputun commented May 14, 2019

webkonstantin commented May 14, 2019

umputun commented May 14, 2019

webkonstantin commented May 14, 2019

Reeywhaar commented May 15, 2019 • edited Loading

umputun commented May 15, 2019

webkonstantin commented Jan 4, 2019 •

edited

Loading

umputun commented May 13, 2019 •

edited

Loading

umputun commented May 14, 2019 •

edited

Loading

Reeywhaar commented May 15, 2019 •

edited

Loading