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

凡例が表示されなくなってしまったのを修正する #379

Merged
merged 1 commit into from
Mar 4, 2021

Conversation

yuiseki
Copy link
Collaborator

@yuiseki yuiseki commented Mar 4, 2021

概要

動作確認

  • yarn run dev して動くことを確認した

スクリーンショット

After

Image from Gyazo

Before

Image from Gyazo

li.legend-item(v-for='(setting, name) in map_config.layer_settings' v-if="displayMarkersGroupByCategory.some((elm) => elm.name === name)")
span.legend-mark(:style="{backgroundColor:setting.color}" @click="selectCategory(name), isOpenList=name, isDisplayAllCategory=false" :class='{open: isDisplayAllCategory || activeCategory === name}')
li.legend-item(v-for='(setting, category) in map_config.layer_settings' v-if="displayMarkersGroupByCategory.some((elm) => elm.category === category)")
span.legend-mark(:style="{backgroundColor:setting.color}" @click="selectCategory(category), isOpenList=category, isDisplayAllCategory=false" :class='{open: isDisplayAllCategory || activeCategory === category}')
Copy link
Contributor

@silloi silloi Mar 4, 2021

Choose a reason for hiding this comment

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

[IMO]なんかここゴチャゴチャっとしているのでtemplateから外出ししたいですね

@click="selectCategory(category), isOpenList=category, isDisplayAllCategory=false"

=>

@click="openCategory(category)"

+追加

    openCategory (category) {
      this.selectCategory(category)
      this.isOpenList = category
      this.isDisplayAllCategory = false
    },

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

このへん読んでても地獄なのでPull Request歓迎です!!!!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

とりあえずissueを立てるとよさそう

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

#380 立てました

Copy link
Contributor

Choose a reason for hiding this comment

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

別PRで対応了解です!

Copy link
Contributor

@silloi silloi left a comment

Choose a reason for hiding this comment

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

LGTM!

@yuiseki yuiseki merged commit 43361b4 into master Mar 4, 2021
@yuiseki yuiseki deleted the fix/378 branch March 4, 2021 09:13
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