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

style: Improve styles for bulk annotation bar and spaces #2356

Conversation

leiyre
Copy link
Member

@leiyre leiyre commented Feb 16, 2023

Description

Improve styles for bulk annotation bar and spaces

Closes #2354

Type of change

(Please delete options that are not relevant. Remember to title the PR according to the type of change)

  • New feature (non-breaking change which adds functionality)

Checklist

  • I have merged the original branch into my forked branch
  • follows the style guidelines of this project
  • I did a self-review of my code
  • My changes generate no new warnings

@leiyre leiyre changed the title styles: Improve styles for bulk annotation bar and spaces style: Improve styles for bulk annotation bar and spaces Feb 16, 2023
@@ -18,14 +18,18 @@
<template>
<div v-if="dataset && viewSettings">
<div
:class="annotationEnabled ? 'list__item--annotation-mode' : 'list__item'"
:class="
annotationEnabled && !pageSizeisOne
Copy link
Contributor

Choose a reason for hiding this comment

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

1/ add a computed for annotationEnabled && !pageSizeisOne
2/ suggestion because it's a boolean : pageSizeisOne => isPageSizeEqualOne (camel case syntax means all the words start with uppercase)

>
<div class="record__header">
<template v-if="!weakLabelingEnabled">
<template v-if="annotationEnabled">
<div class="record__header--left" v-if="!isReferenceRecord">
<base-checkbox
:class="pageSizeisOne && '--hidden'"
v-if="!pageSizeisOne"
Copy link
Contributor

Choose a reason for hiding this comment

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

update pageSizeisOne here too

@leiyre leiyre merged commit 81d27b4 into feature/bulk-annotation-improvement Feb 17, 2023
@leiyre leiyre deleted the feat/styles_improvement_bulk_and_spaces branch February 17, 2023 14:03
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