Skip to content
  • This project
    • Loading...
  • Sign in

Yokihito Oki / kanoya-univercity-prod

Go to a project
Toggle navigation
Toggle navigation pinning
  • Projects
  • Groups
  • Snippets
  • Help
  • Project
  • Activity
  • Repository
  • Pipelines
  • Graphs
  • Issues 0
  • Merge Requests 0
  • Wiki
  • Network
  • Create a new issue
  • Builds
  • Commits
  • Issue Boards
Merged
Merge Request !10 opened 2020-08-04 12:25:15 +0900 by TaishiTokudome@tokudome

インタビューページのスタイル指定

Edited 2020-10-22 13:06:45 +0900
Request to merge interview_dev into master

Merged

The changes were merged into master. The source branch has been removed.

  • Discussion 17
  • Commits 26
  • Changes 4
  • {{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • TaishiTokudome
    @tokudome

    added 61 commits

    • 2156a3a9...ebd2d78b - 60 commits from branch master
    • c41f7a31 - コンフリクト解消

    Compare with previous version

    2020-09-01 09:36:27 +0900

    added 61 commits

    • 2156a3a9...ebd2d78b - 60 commits from branch master
    • c41f7a31 - コンフリクト解消

    Compare with previous version

    Toggle commit list
  • TaishiTokudome
    @tokudome

    added 2 commits

    • d7c66042 - サムネイル付き記事一覧のスタイル指定
    • 22d5103c - イントロ画像登録時の画像リサイズ処理

    Compare with previous version

    2020-09-01 13:08:04 +0900

    added 2 commits

    • d7c66042 - サムネイル付き記事一覧のスタイル指定
    • 22d5103c - イントロ画像登録時の画像リサイズ処理

    Compare with previous version

    Toggle commit list
  • Toggle discussion
    Yokihito Oki
    @oki started a discussion on an outdated diff 2020-09-01 14:07:46 +0900
    Last updated by TaishiTokudome 2020-09-01 14:43:46 +0900
    components/com_jce/editor/libraries/classes/browser.php
    1173 1173
    1174 1174 // get uploaded file
    1175 1175 $file = $app->input->files->get('file', array(), 'raw');
    1176
    1176 $file['size'] = 40;
    1177 // error_log("====================================================================================================================\n", 3, '/var/www/html/debug.log');
    • Yokihito Oki
      @oki commented 2020-09-01 14:07:46 +0900
      Master

      不要なら削除推奨

    • TaishiTokudome
      @tokudome commented 2020-09-01 14:43:46 +0900
      Master

      不要ですので削除します。

      Edited 2020-09-01 14:51:16 +0900
  • Toggle discussion
    Yokihito Oki
    @oki started a discussion on an outdated diff 2020-09-01 14:08:06 +0900
    Last updated by TaishiTokudome 2020-09-01 14:43:55 +0900
    components/com_jce/editor/libraries/classes/browser.php
    1175 1175 $file = $app->input->files->get('file', array(), 'raw');
    1176
    1176 $file['size'] = 40;
    1177 // error_log("====================================================================================================================\n", 3, '/var/www/html/debug.log');
    1178 // error_log("outputed from " . __FILE__ . "\n", 3, '/var/www/html/debug.log');
    1179 // error_log('file = ', 3, '/var/www/html/debug.log');
    1180 // error_log(print_r($file, true) . "\n", 3, '/var/www/html/debug.log');
    1181 // error_log("====================================================================================================================\n", 3, '/var/www/html/debug.log');
    1182 // error_log("ここまで\n\n", 3, '/var/www/html/debug.log');
    1177 1183 // validate file
    1178 1184 $this->validateUploadedFile($file);
    1179 1185
    1180 1186 // get file name
    1181 1187 $name = (string) $app->input->get('name', $file['name'], 'STRING');
    1182
    1188 // error_log("====================================================================================================================\n", 3, '/var/www/html/debug.log');
    • Yokihito Oki
      @oki commented 2020-09-01 14:08:06 +0900
      Master

      不要なら削除推奨

    • TaishiTokudome
      @tokudome commented 2020-09-01 14:43:55 +0900
      Master

      不要ですので削除します。

  • Toggle discussion
    Yokihito Oki
    @oki started a discussion on an outdated diff 2020-09-01 14:15:20 +0900
    Last updated by TaishiTokudome 2020-09-01 14:48:14 +0900
    components/com_jce/editor/libraries/classes/browser.php
    1239 1250 }
    1240 1251 }
    1252 /**
    1253 * 画像リサイズ処理
    1254 */
    1255 $fieldid = (string) $app->input->get('fieldid', '', 'STRING');
    1256 // ファイルがイントロ画像としてアップロードされていることを確認
    1257 if ($fieldid === 'jform_images_image_intro') {
    1258 // tmpディレクトリに一時保存されているupload予定のファイルパスを取得
    1259 $tmp_file = $file['tmp_name'];
    1260 // 画像を取得
    1261 $img = file_get_contents($tmp_file);
    1262 // 画像のエンコード
    1263 $enc_img = base64_encode($img);
    1264 // 画像URLを作成
    1265 $image_url = 'data:application/octet-stream;base64,' . $enc_img;
    • Yokihito Oki
      @oki commented 2020-09-01 14:15:20 +0900
      Master

      URLでなくデータ??

    • TaishiTokudome
      @tokudome commented 2020-09-01 14:47:26 +0900
      Master

      Base64エンコードを理解していませんでした。 Base64エンコードは64種類の英数字のみを使い、それ以外の文字を扱うことの出来ない環境でデータを扱うためのエンコード方式であり、 今回はBase64エンコードをしなくても画像を扱えることを確認したので、処理を以下のように書き換えます

      Edited 2020-09-01 14:49:27 +0900
    • TaishiTokudome
      @tokudome commented 2020-09-01 14:48:14 +0900
      Master
      // tmpディレクトリに一時保存されているupload予定のファイルパスを取得
      $tmp_file = $file['tmp_name'];
      
      // 画像情報を取得
      $image_info = getimagesize($tmp_file);
      
      Edited 2020-09-01 14:48:44 +0900
  • Toggle discussion
    Yokihito Oki
    @oki started a discussion on an outdated diff 2020-09-01 14:24:39 +0900
    Last updated by TaishiTokudome 2020-09-01 16:47:30 +0900
    components/com_jce/editor/libraries/classes/browser.php
    1261 $img = file_get_contents($tmp_file);
    1262 // 画像のエンコード
    1263 $enc_img = base64_encode($img);
    1264 // 画像URLを作成
    1265 $image_url = 'data:application/octet-stream;base64,' . $enc_img;
    1266
    1267 // 画像情報を取得
    1268 $image_info = getimagesize($image_url);
    1269 // 画像情報から画像のサイズを取得
    1270 $width = (int) $image_info[0];
    1271 $height = (int) $image_info[1];
    1272 // 画像タイプを取得
    1273 $image_type = $image_info['mime'];
    1274 // アスペクト比(縦横比を取得)
    1275 $aspect_ratio = $height / $width;
    1276
    • Yokihito Oki
      @oki commented 2020-09-01 14:24:39 +0900
      Master

      元画像のwidthが1000px以下の場合、引き伸ばす仕様ってことですけ? リサイズの仕様を詰めた方が良いかと。

    • TaishiTokudome
      @tokudome commented 2020-09-01 16:47:30 +0900
      Master

      リサイズの仕様ですがもう少し検討が必要ですので、次回commit時に仕様を固めて、その修正をまとめてcommitします。

  • Toggle discussion
    Yokihito Oki
    @oki started a discussion on the diff 2020-09-01 14:35:07 +0900
    Last updated by TaishiTokudome 2020-09-01 15:17:51 +0900
    components/com_jce/editor/libraries/classes/browser.php
    1284 // 画像タイプがPNGの場合の処理
    1285 if ($image_type === 'image/png') {
    1286 // 元の画像から新しい画像を作る準備
    1287 $baseImage = imagecreatefrompng($image_url);
    1288 // 画像タイプがJPEGの場合の処理
    1289 } elseif ($image_type === 'image/jpeg') {
    1290 // 元の画像から新しい画像を作る準備
    1291 $baseImage = imagecreatefromjpeg($image_url);
    1292 }
    1293 // サイズを指定して新しい画像のキャンバスを作成
    1294 $image = imagecreatetruecolor($new_width, $new_height);
    1295 // 画像のコピーと伸縮
    1296 imagecopyresampled($image, $baseImage, 0, 0, 0, 0, $new_width, $new_height, $width, $height);
    1297 // コピーした画像を出力
    1298 imagejpeg($image , $tmp_file);
    1299 }
    • Yokihito Oki
      @oki commented 2020-09-01 14:35:07 +0900
      Master

      条件が2つなんでこれでも問題ないけど、ネストが深いのが気になる。 条件増えるならswitchのがよいかも。 $baseImage = NULL; switch ($image_type) { case 'image/png': $baseImage = imagecreatefrompng($image_url); break; case 'image/jpeg': $baseImage = imagecreatefromjpeg($image_url); break; default: break; } if ($baseImage) { $image = imagecreatetruecolor($new_width, $new_height); imagecopyresampled($image, $baseImage, 0, 0, 0, 0, $new_width, $new_height, $width, $height); imagejpeg($image , $tmp_file); }

      Edited 2020-09-01 14:36:02 +0900
    • TaishiTokudome
      @tokudome commented 2020-09-01 15:17:51 +0900
      Master

      勉強になります。 switch構文による処理の分岐に書き換えました。

  • Toggle discussion
    Yokihito Oki
    @oki started a discussion on an outdated diff 2020-09-01 14:37:15 +0900
    Last updated by TaishiTokudome 2020-09-01 14:50:25 +0900
    templates/protostar/css/style_org.css
    3439 flex-direction: row-reverse;
    3440 justify-content: space-between;
    3441 align-items: center;
    3442 }
    3443
    3444 /* イントロテキストを表示しない設定ができないので、以下の設定で、一旦全てを非表示にする */
    3445 .interview div.items-leading > div > * {
    3446 display: none;
    3447 }
    3448
    3449 .interview div.items-leading > div > div {
    3450 display: block;
    3451 }
    3452
    3453 .interview div.items-leading div div.page-header {
    3454 width: 100%;
    • Yokihito Oki
      @oki commented 2020-09-01 14:37:15 +0900
      Master

      インデントがおかしい

    • TaishiTokudome
      @tokudome commented 2020-09-01 14:50:25 +0900
      Master

      インデント修正します

  • Toggle discussion
    Yokihito Oki
    @oki started a discussion on an outdated diff 2020-09-01 14:41:18 +0900
    Last updated by TaishiTokudome 2020-09-01 16:42:46 +0900
    templates/protostar/css/style_org.css
    3468 .interview div.items-leading div div.item-image img {
    3469 /* 画像ごとによって縦横幅が自動設定。最大値以上にはならないように指定 */
    3470 width: 100%;
    3471 }
    3472
    3473 .interview div.items-leading div div.item-image {
    3474 margin: 20px 20px 20px 0;
    3475 }
    3476
    3477 /* レスポンシブ */
    3478 @media (max-width: 500px) {
    3479 .interview div.items-leading > div {
    3480 flex-direction: column-reverse;
    3481 }
    3482
    3483 .interview div.items-leading div div.page-header h2 {
    • Yokihito Oki
      @oki commented 2020-09-01 14:41:18 +0900
      Master

      このセレクタは二宮流?? 意図的なら良いけど、柔軟性が低い。

    • TaishiTokudome
      @tokudome commented 2020-09-01 16:42:46 +0900
      Master

      意図的に要素タグをセレクタとして指定する箇所以外は、クラスをセレクタとして指定します。

  • TaishiTokudome
    @tokudome

    added 1 commit

    • caeb56f3 - インタビューページのスタイル修正

    Compare with previous version

    2020-09-02 08:16:26 +0900

    added 1 commit

    • caeb56f3 - インタビューページのスタイル修正

    Compare with previous version

    Toggle commit list
  • TaishiTokudome
    @tokudome

    added 2 commits

    • b36c9d9d - 画像リサイズ処理の修正
    • dff254c0 - インタビューページのスタイル修正

    Compare with previous version

    2020-09-02 13:41:49 +0900

    added 2 commits

    • b36c9d9d - 画像リサイズ処理の修正
    • dff254c0 - インタビューページのスタイル修正

    Compare with previous version

    Toggle commit list
  • TaishiTokudome
    @tokudome

    added 1 commit

    • e6f1a889 - インタビューページのスタイル修正

    Compare with previous version

    2020-09-02 13:54:45 +0900

    added 1 commit

    • e6f1a889 - インタビューページのスタイル修正

    Compare with previous version

    Toggle commit list
  • Toggle discussion
    Yokihito Oki
    @oki started a discussion on an outdated diff 2020-09-02 16:46:57 +0900
    components/com_jce/editor/libraries/classes/browser.php
    1272 // 画像タイプがPNGの場合の処理
    1273 case 'image/png':
    1274 // 元の画像から新しい画像を作る準備
    1275 $base_image = imagecreatefrompng($tmp_file);
    1276 break;
    1277 // 画像タイプがJPEGの場合の処理
    1278 case 'image/jpeg':
    1279 // 元の画像から新しい画像を作る準備
    1280 $base_image = imagecreatefromjpeg($tmp_file);
    1281 break;
    1282 default:
    1283 break;
    1284 }
    1285
    1286 // $base_imageがnullから更新されている場合を確認
    1287 if ($base_image) {
    • Yokihito Oki
      @oki commented 2020-09-02 16:46:57 +0900
      Master

      if (!is_null($base_image)) { ... }

  • Toggle discussion
    Yokihito Oki
    @oki started a discussion on an outdated diff 2020-09-02 17:04:42 +0900
    templates/protostar/css/style_org.css
    3437 border-top: 5px solid #003894;
    3438 display: flex;
    3439 flex-direction: row-reverse;
    3440 align-items: center;
    3441 position: relative;
    3442 }
    3443
    3444 /* イントロテキストを表示しない設定ができないので、以下の設定で、一旦全てを非表示にする */
    3445 .interview .items-leading > div > * {
    3446 display: none;
    3447 }
    3448
    3449 .interview .items-leading div .page-header {
    3450 display: block;
    3451 width: 100%;
    3452 max-width: 800px;
    • Yokihito Oki
      @oki commented 2020-09-02 17:04:42 +0900
      Master

      必要?

  • Yokihito Oki
    @oki

    added 2 commits

    • 3ae507d6 - インタビューページのスタイル修正
    • cd407d9c - 画像リサイズ処理の修正

    Compare with previous version

    2020-09-02 18:22:17 +0900

    added 2 commits

    • 3ae507d6 - インタビューページのスタイル修正
    • cd407d9c - 画像リサイズ処理の修正

    Compare with previous version

    Toggle commit list
  • TaishiTokudome
    @tokudome

    added 2 commits

    • 0066723c - リサイズ処理の修正
    • db04547e - インタビューページのスタイル修正

    Compare with previous version

    2020-09-03 10:58:09 +0900

    added 2 commits

    • 0066723c - リサイズ処理の修正
    • db04547e - インタビューページのスタイル修正

    Compare with previous version

    Toggle commit list
  • TaishiTokudome
    @tokudome

    added 5 commits

    • db04547e...52365c65 - 4 commits from branch master
    • 4b41f4e1 - Merge branch 'master' into 'interview_dev'

    Compare with previous version

    2020-09-27 20:09:28 +0900

    added 5 commits

    • db04547e...52365c65 - 4 commits from branch master
    • 4b41f4e1 - Merge branch 'master' into 'interview_dev'

    Compare with previous version

    Toggle commit list
  • Yokihito Oki
    @oki

    added 1 commit

    • 0cad6fb1 - インタビューページタイトル スタイル指定

    Compare with previous version

    2020-10-08 12:05:40 +0900

    added 1 commit

    • 0cad6fb1 - インタビューページタイトル スタイル指定

    Compare with previous version

    Toggle commit list
  • Yokihito Oki
    @oki

    added 1 commit

    • e216879a - インタビュー記事 記事装飾クラスのスタイル指定

    Compare with previous version

    2020-10-08 12:47:47 +0900

    added 1 commit

    • e216879a - インタビュー記事 記事装飾クラスのスタイル指定

    Compare with previous version

    Toggle commit list
  • TaishiTokudome
    @tokudome

    added 10 commits

    • c81ee506 - ビデオライブラリページのスタイル指定
    • 774d32cc - ビデオライブラリページのスタイル修正
    • 892c55b8 - Merge branch 'interview_dev' into 'video_liblary'
    • 6e927f3c - フォトライブラリ機能の追加
    • 6f6ff299 - フォトライブラリ スタイル修正
    • d5c7baee - フォトライブラリ スタイル修正
    • eb678b6a - フォトライブラリページの修正
    • 8d94c230 - フォトライブラリページの修正
    • 15efb870 - フォトライブラリページの修正
    • c3a512ab - Merge branch 'video_liblary' into 'interview_dev'

    Compare with previous version

    2020-10-15 09:15:01 +0900

    added 10 commits

    • c81ee506 - ビデオライブラリページのスタイル指定
    • 774d32cc - ビデオライブラリページのスタイル修正
    • 892c55b8 - Merge branch 'interview_dev' into 'video_liblary'
    • 6e927f3c - フォトライブラリ機能の追加
    • 6f6ff299 - フォトライブラリ スタイル修正
    • d5c7baee - フォトライブラリ スタイル修正
    • eb678b6a - フォトライブラリページの修正
    • 8d94c230 - フォトライブラリページの修正
    • 15efb870 - フォトライブラリページの修正
    • c3a512ab - Merge branch 'video_liblary' into 'interview_dev'

    Compare with previous version

    Toggle commit list
  • TaishiTokudome
    @tokudome

    added 1 commit

    • dc16b074 - インタビューページのCSS修正

    Compare with previous version

    2020-10-15 09:16:37 +0900

    added 1 commit

    • dc16b074 - インタビューページのCSS修正

    Compare with previous version

    Toggle commit list
  • TaishiTokudome
    @tokudome

    merged

    2020-10-22 13:06:45 +0900

    merged

    Toggle commit list
  • TaishiTokudome
    @tokudome

    mentioned in commit 58f2c7b1

    2020-10-22 13:06:45 +0900

    mentioned in commit 58f2c7b1

    Toggle commit list
  • Write
  • Preview
Styling with Markdown is supported
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
  • Please register or sign in to post a comment
Assignee
No assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
2
2 participants
Reference: oki/kanoya-univercity-prod!10
×

Revert this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.
×

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.