fix: invalid toggleMode option in full mobile panel

lijinke666 · Sep 26, 2020 · da20e32 · da20e32
1 parent 576470f
commit da20e32
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/__tests__/tests/player.test.js b/__tests__/tests/player.test.js
@@ -1495,4 +1495,12 @@ describe('<ReactJkMusicPlayer/>', () => {
     wrapper.find('.hide-panel').simulate('click')
     expect(onAudioListsPanelChange).not.toHaveBeenCalled()
   })
+
+  it('should not render toggle mode close button in mobile panel', () => {
+    const wrapper = mount(<ReactJkMusicPlayer mode="full" toggleMode={false} />)
+    wrapper.setState({ isMobile: true })
+    expect(
+      wrapper.find('.react-jinke-music-player-mobile-header-right'),
+    ).toHaveLength(0)
+  })
 })
diff --git a/src/components/PlayerMobile.js b/src/components/PlayerMobile.js
@@ -28,6 +28,7 @@ const PlayerMobile = ({
   autoHiddenCover,
   icon,
   locale,
+  toggleMode,
 }) => (
   <div className={cls(prefix, { 'default-bg': !glassBg, 'glass-bg': glassBg })}>
     <PlayModeTip
@@ -41,9 +42,11 @@ const PlayerMobile = ({
       <div className={`${prefix}-header-title`} title={name}>
         {name}
       </div>
-      <div className={`${prefix}-header-right`} onClick={onClose}>
-        {icon.close}
-      </div>
+      {toggleMode && (
+        <div className={`${prefix}-header-right`} onClick={onClose}>
+          {icon.close}
+        </div>
+      )}
     </div>
     <div className={`${prefix}-singer text-center group`}>
       <span className={`${prefix}-singer-name`} title={singer}>

diff --git a/src/index.js b/src/index.js
@@ -485,6 +485,7 @@ export default class ReactJkMusicPlayer extends PureComponent {
             autoHiddenCover={autoHiddenCover}
             onCoverClick={this.onCoverClick}
             locale={locale}
+            toggleMode={toggleMode}
           />
         )}