firefish
/
firefish
27
340
Fork 100
Code Issues Pull Requests Actions Projects 3 Releases 809 Activity

Add nyaize syntax #3980

Closed
acid-chicken wants to merge 5 commits from nya into develop
pull from: nya
merge into: firefish:develop
Conversation 7 Commits 5 Files Changed 3
acid-chicken commented 2019-01-24 10:10:33 +00:00 (Migrated from github.com)

Summary

Resolves #3974

Syntax

It's not mfm syntax.

  • <nya> -> starts nyaize enable zone
  • </nya> -> ends nyaize enable zone
  • <!nya> -> starts nyaize disable zone
  • </!nya> -> ends nyaize disable zone
Summary == Resolves #3974 Syntax -- It's not mfm syntax. * `<nya>` -> starts nyaize enable zone * `</nya>` -> ends nyaize enable zone * `<!nya>` -> starts nyaize disable zone * `</!nya>` -> ends nyaize disable zone
acid-chicken commented 2019-01-24 10:28:30 +00:00 (Migrated from github.com)

Should add some tests.

Should add some tests.
AyaMorisawa commented 2019-01-24 10:32:39 +00:00 (Migrated from github.com)

@acid-chicken Please clarify your solution to the issue so that we can review your PR.

@acid-chicken Please clarify your solution to the issue so that we can review your PR.
syuilo commented 2019-01-24 10:34:49 +00:00 (Migrated from github.com)

#3974 を解決する目的なら、サーバー側でMFM解析した上で検索構文やハッシュタグ以外のところだけnyaizeするようにした方がシンプルそう

#3974 を解決する目的なら、サーバー側でMFM解析した上で検索構文やハッシュタグ以外のところだけnyaizeするようにした方がシンプルそう
acid-chicken commented 2019-01-24 10:40:07 +00:00 (Migrated from github.com)

@syuilo 一部の場所だけ暗黙的にnyanizeが解除されたりするのはちょっと気持ち悪そうというのがあります。

@syuilo 一部の場所だけ暗黙的にnyanizeが解除されたりするのはちょっと気持ち悪そうというのがあります。
😕 1
AyaMorisawa commented 2019-01-24 10:43:32 +00:00 (Migrated from github.com)

#3974 を解決する目的なら、サーバー側でMFM解析した上で検索構文やハッシュタグ以外のところだけnyaizeするようにした方がシンプルそう

サーバー側で解析する必要ありますか？クライアント側では不十分ですか？

> #3974 を解決する目的なら、サーバー側でMFM解析した上で検索構文やハッシュタグ以外のところだけnyaizeするようにした方がシンプルそう サーバー側で解析する必要ありますか？クライアント側では不十分ですか？
syuilo commented 2019-01-24 10:45:22 +00:00 (Migrated from github.com)

クライアントでもいいかもしれませんがクライアントが対応してないとnyaizeされなくなってしまう点が微妙
(Cat機能は問答無用で な が にゃ になってしまう理不尽さを楽しむジョーク機能なので)

クライアントでもいいかもしれませんがクライアントが対応してないとnyaizeされなくなってしまう点が微妙 (Cat機能は問答無用で な が にゃ になってしまう理不尽さを楽しむジョーク機能なので)
acid-chicken commented 2019-02-10 10:51:55 +00:00 (Migrated from github.com)

(Cat機能は問答無用で な が にゃ になってしまう理不尽さを楽しむジョーク機能なので)

なので知る人ぞ知る隠し構文みたいな感じでしれっと実装できちゃったりしませんか。

> (Cat機能は問答無用で な が にゃ になってしまう理不尽さを楽しむジョーク機能なので) なので知る人ぞ知る隠し構文みたいな感じでしれっと実装できちゃったりしませんか。

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
1️⃣ Good First Issue
   
❓needs more investigation

A bug whose causes are unknown   
Accessibility

Issues and PRs about improving accessibility   
Antennas

The Antennas feature   
Channels

The Channels feature   
Chat

The Chat feature   
Clips

The Clips feature   
CP/Settings

Issues about user or admin settings   
dependencies

Pull requests that update a dependency file   
Docker

PRs and Issues relating to docker   
Drive

The Drive feature   
External

Bugs caused by external libraries   
Galleries

The Galleries feature   
Groups

The Groups feature   
i18n

Internationalization (i18n) related issue/PR   
javascript

Pull requests that update Javascript code   
Kubernetes

Issues and PRs relating to Kubernetes   
Lists/Circles

The Lists/Circles feature   
Mobile

Issues about using Calckey with a mobile device   
Mutes/Blocks

Issues about mutes, blocks, silence   
Notifications

Issues about notifications   
Posts/Timelines

Issues about posts, timelines, threads   
Widgets

The Widgets feature   
‼️ wrong locales

This PR edits locales other than the ja-JP one. See locales/README.md   
☢️Breaking

This change breaks compatibility   
♻️Duplicate

This issue/PR already exists   
⚙️Server

Server side specific issue/PR   
⚠️bug?

This might be a bug   
✨Feature

This adds/improves/enhances a feature   
🧩API

Interface between server and client   
🐛Bug

Unexpected behavior   
💬Discussion

Being discussed or needs discussion   
📖Doc

Documentation related issue/PR   
🌌Federation

The Federation feature   
💴has reward
   
🔥high priority
   
🚫Invalid

Not based on facts / Outside the scope of Calckey   
🙏low priority
   
🖍MFM

The Misskey Flavored Markdown feature   
🧪needs test

This needs test code   
📜Pages

The Pages feature   
🐢Performance

Efficiency related issue/PR   
💚Refactor

Rewriting code without changing behavior   
🏠Room

The Room feature   
🔒Security

Security related issue/PR   
🤖Service Worker

SW related issue/PR   
🧪Test

Test related issue/PR   
🚧WIP

This is a work in progress   
🗿Wontfix

This will not be worked on   
👩‍💻AiScript

Issues/PRs related to AiScript   
🖥️Client

Client side specific issue/PR   
🛠️Dev

Development of Calckey itself   
🏳️needs help

This needs extra attention
No Label
1️⃣ Good First Issue
❓needs more investigation
Accessibility
Antennas
Channels
Chat
Clips
CP/Settings
dependencies
Docker
Drive
External
Galleries
Groups
i18n
javascript
Kubernetes
Lists/Circles
Mobile
Mutes/Blocks
Notifications
Posts/Timelines
Widgets
‼️ wrong locales
☢️Breaking
♻️Duplicate
⚙️Server
⚠️bug?
✨Feature
🧩API
🐛Bug
💬Discussion
📖Doc
🌌Federation
💴has reward
🔥high priority
🚫Invalid
🙏low priority
🖍MFM
🧪needs test
📜Pages
🐢Performance
💚Refactor
🏠Room
🔒Security
🤖Service Worker
🧪Test
🚧WIP
🗿Wontfix
👩‍💻AiScript
🖥️Client
🛠️Dev
🏳️needs help
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: firefish/firefish#3980
There is no content yet.
Delete Branch "nya"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?